-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Blank Sigma Visualization (on AUK) #26
Comments
Weird. Wonder why it is producing |
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. |
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. |
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. |
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). |
What's producing the 1.1 files? iGraph or GraphPass? |
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. |
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. |
Yep the change happened here. |
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. |
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). |
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. |
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. |
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. |
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. |
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 |
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.
The text was updated successfully, but these errors were encountered: