Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upBlank Sigma Visualization (on AUK) #26
Comments
ianmilligan1
added
the
bug
label
Aug 24, 2018
ianmilligan1
assigned
greebie
Aug 24, 2018
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ruebot
Aug 24, 2018
Member
Weird. Wonder why it is producing <?xml version="1.1" encoding="UTF-8"?>
instead of <?xml version="1.0" encoding="UTF-8"?>
. The gexf files that display fine are all 1.0.
Weird. Wonder why it is producing |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ianmilligan1
Aug 24, 2018
Member
Heh I was just looking at that (since the xml linter I have only supports 1.1). That said, the Halifax Explosion GEXF file that I have (the file is here for downloading) and which works is also 1.1.
Heh I was just looking at that (since the xml linter I have only supports 1.1). That said, the Halifax Explosion GEXF file that I have (the file is here for downloading) and which works is also 1.1. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ianmilligan1
Aug 24, 2018
Member
OK if you manually change <?xml version="1.1" encoding="UTF-8"?>
to <?xml version="1.0" encoding="UTF-8"?>
in a text editor, it renders in sigma js.
But in this case, the error documented in #25 appears (see screenshot below) – so these two issues are probably related.
OK if you manually change But in this case, the error documented in #25 appears (see screenshot below) – so these two issues are probably related. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
greebie
Aug 27, 2018
Collaborator
I think the problem is related to #4 in this article -- you have to use \x0A
instead of \n
for XML 1.1 line breaks.
After my research, my opinion is that 1.1 is not worth supporting. 1.0 covers the vast majority of use cases and does not create problems like 1.1 does.
I think the problem is related to #4 in this article -- you have to use After my research, my opinion is that 1.1 is not worth supporting. 1.0 covers the vast majority of use cases and does not create problems like 1.1 does. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ianmilligan1
Aug 27, 2018
Member
OK, agreed, XML 1.0 would be ideal I think – XML 1.1 throws up errors and I think there aren't too many linters/etc. that work out of the box with it (that said, I'm not really an XML guy).
OK, agreed, XML 1.0 would be ideal I think – XML 1.1 throws up errors and I think there aren't too many linters/etc. that work out of the box with it (that said, I'm not really an XML guy). |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
What's producing the 1.1 files? iGraph or GraphPass? |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
greebie
Aug 27, 2018
Collaborator
Looks like GraphPass when it creates the Gexf. Don't know why I added that. Might be when I tried to switch to Gexf 1.3.
Looks like GraphPass when it creates the Gexf. Don't know why I added that. Might be when I tried to switch to Gexf 1.3. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ruebot
Aug 27, 2018
Member
Ok. We need to be very cognizant of breaking changes like this. If there are any major version changes that effect things downstream in the stack, we need to talk about them. Not just implement them.
Ok. We need to be very cognizant of breaking changes like this. If there are any major version changes that effect things downstream in the stack, we need to talk about them. Not just implement them. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Yep the change happened here. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
greebie
Aug 27, 2018
Collaborator
Gexf 1.2 is deprecated. Looking at the traceback on the Gexf 1.3 issue, there were requests to upgrade to 1.1. https://github.com/gephi/gexf/wiki
It's possible switching back may not fit the XML schema, but I'll check on this tomorrow.
Gexf 1.2 is deprecated. Looking at the traceback on the Gexf 1.3 issue, there were requests to upgrade to 1.1. https://github.com/gephi/gexf/wiki It's possible switching back may not fit the XML schema, but I'll check on this tomorrow. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ianmilligan1
Aug 27, 2018
Member
Can you link to the GEXF 1.3 issue that you mentioned above?
Presumably we can revert some of the changes that appeared in this commit?
In any case, if we're doing changes like GEXF 1.2 -> 1.3 or XML 1.0 -> 1.1, in the future we can't have that sort of stuff sneaking into pull requests like that one above (which was solving the 50k node cut off value).
Can you link to the GEXF 1.3 issue that you mentioned above? Presumably we can revert some of the changes that appeared in this commit? In any case, if we're doing changes like GEXF 1.2 -> 1.3 or XML 1.0 -> 1.1, in the future we can't have that sort of stuff sneaking into pull requests like that one above (which was solving the 50k node cut off value). |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
greebie
Aug 27, 2018
Collaborator
It was not an issue here on Graphpass, but on the switch on the Gephi repo from 1.2 -> 1.3.
It's possible that this was causing headaches for me, since Graphpass originated as a personal project for filtering networks in different ways.
It may make sense in future to take the components necessary for AUK and move them to a separate library.
It was not an issue here on Graphpass, but on the switch on the Gephi repo from 1.2 -> 1.3. It's possible that this was causing headaches for me, since Graphpass originated as a personal project for filtering networks in different ways. It may make sense in future to take the components necessary for AUK and move them to a separate library. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ianmilligan1
Aug 27, 2018
Member
Can you still link to issues when you mention them so we know what’s going on? :)
That might make sense as we are trying to use this in a production environment. In any case we need to get this to work with AUK again.
Can you still link to issues when you mention them so we know what’s going on? :) That might make sense as we are trying to use this in a production environment. In any case we need to get this to work with AUK again. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
greebie
Aug 27, 2018
Collaborator
Better yet - when I do personal / research project changes, I will work from a fork starting now. Graphpass can be formalized into a proper AUT project.
Better yet - when I do personal / research project changes, I will work from a fork starting now. Graphpass can be formalized into a proper AUT project. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
greebie
Aug 27, 2018
Collaborator
I don't know - I just know that I get a 1.2 deprecation warning when I use Gephi. I added an issue to the Gexf repo to see if I can get some clarity.
I don't know - I just know that I get a 1.2 deprecation warning when I use Gephi. I added an issue to the Gexf repo to see if I can get some clarity. |
added a commit
to ruebot/graphpass
that referenced
this issue
Aug 27, 2018
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ianmilligan1
Aug 28, 2018
Member
FWIW I tested @ruebot's branch on the Scarborough collection. Less errors on load so I think it's fixed (I'm testing on some UVic collections right now). But it still displayed a blank canvas.
I think that is because in the case of Scarborough (75/5715), this snuck in to the GEXF:
<node id="n0" label="wix.com">
<attvalues>
<attvalue for="v_label" value="wix.com" />
<attvalue for="v_id" value="d2a067fb67354846222e462d124233e0" />
<attvalue for="v_Degree" value="1" />
<attvalue for="v_WalkTrapModularity" value="0" />
<attvalue for="v_r" value="35" />
<attvalue for="v_g" value="217" />
<attvalue for="v_b" value="211" />
<attvalue for="v_x" value="-112.597" />
<attvalue for="v_y" value="86.8064" />
</attvalues>
<viz:color r="35" g="217" b="211"></viz:color>
<viz:size value="nan"></viz:size>
<viz:position y="86.806363" x="-112.596639" z="0.0"></viz:position>
</node>
<node id="n1" label="youtube.com">
<attvalues>
<attvalue for="v_label" value="youtube.com" />
<attvalue for="v_id" value="14dd5266c70789bdc806364df4586335" />
<attvalue for="v_Degree" value="1" />
<attvalue for="v_WalkTrapModularity" value="0" />
<attvalue for="v_r" value="35" />
<attvalue for="v_g" value="217" />
<attvalue for="v_b" value="211" />
<attvalue for="v_x" value="119.755" />
<attvalue for="v_y" value="20.5159" />
</attvalues>
<viz:color r="35" g="217" b="211"></viz:color>
<viz:size value="nan"></viz:size>
<viz:position y="20.515895" x="119.755066" z="0.0"></viz:position>
</node>
Note that both nodes have a size value of nan
. Probably related to #25?
FWIW I tested @ruebot's branch on the Scarborough collection. Less errors on load so I think it's fixed (I'm testing on some UVic collections right now). But it still displayed a blank canvas. I think that is because in the case of Scarborough (75/5715), this snuck in to the GEXF:
Note that both nodes have a size value of |
ianmilligan1 commentedAug 24, 2018
On some collections, instead of getting a nice interactive Sigma.js diagram, we get a blank graph and a console error claiming that we have some Invalid XML.
This GEXF file works in Gephi but doesn't work in Sigma js (verified by swapping it out with working GEXF files).
From Firefox, here's the whole console error:
How Will This Be Resolved
If we can have the GEXF file linked above work in AUK.