Skip to content
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

add .gexf to allowed files #12

Closed
csae8092 opened this issue Sep 18, 2024 · 6 comments · Fixed by #13
Closed

add .gexf to allowed files #12

csae8092 opened this issue Sep 18, 2024 · 6 comments · Fixed by #13
Assignees
Labels
enhancement New feature or request

Comments

@csae8092
Copy link
Member

https://de.wikipedia.org/wiki/Graph_Exchange_XML_Format

@csae8092 csae8092 self-assigned this Sep 18, 2024
@csae8092 csae8092 added the enhancement New feature or request label Sep 18, 2024
@csae8092 csae8092 linked a pull request Sep 18, 2024 that will close this issue
@zozlak
Copy link
Member

zozlak commented Sep 18, 2024

Oh my, something so popular it has a wikipedia page only in German...

Is this something provided by a depositor?

@zozlak
Copy link
Member

zozlak commented Sep 18, 2024

By the way, the filechecker recognizes it as gexf or as a generic xml?

@zozlak
Copy link
Member

zozlak commented Sep 18, 2024

Last but not least historically adding new accepted formats was controlled by @bellerophons-pegasus . I'm not sure what's the procedure at the moment (@sstuhec ?).

@csae8092
Copy link
Member Author

csae8092 commented Sep 18, 2024

filechecker

{"directory":"/data","filename":"relations.gexf","severity":"ERROR","errorType":"File format not accepted","errorMessage":"MIME application/xml: accepted, extension gexf: not accepted"}

popular:

well, its documented https://gexf.net/ ; supported by Gephi and by network libraries like NetworkX

provided by depositor

yes, but the depositor is me, so I can rename the file

@zozlak
Copy link
Member

zozlak commented Sep 18, 2024

So you have a few options:

  • Make @Seta or @bellerophons-pegasus say yes in which case we will merge your pull request and this will make the filechecker to issue only a warning (cause it will still don't like that much a generic application/xml file; still I can deal with it later by hand-tuning the filechecker to recognize the gexf based on the xml content).
  • Rename your file to .xml which will have same consequences as above but no acceptance is needed from anyone else.
  • Serialize your graph in another format which is recognized by the DROID (the file format recognition tool used by the filechecker) and accepter by the ARCHE. Unfortunately quick research indicates DROID sucks at recognizing graph formats listed on the Gephi supported formats list and I guess when it comes to the ARCHE we currently accept only CSV from that list (and IMHO using a CSV would make it slightly cryptic for reuse). So it doesn't really look like a viable option.

@bellerophons-pegasus
Copy link
Contributor

bellerophons-pegasus commented Sep 19, 2024

We can safely include the format as accepted because:

I would not say it is preferred though. There seem to be better options like GraphML, JSON graph or GML (Graph Modelling Language), which are also supported across a larger selection of network analysis software

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants