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

GDF file format support in readwrite #4979

Closed
wants to merge 7 commits into from

Conversation

stijn-uva
Copy link

This PR adds support for the GDF network file format via the functions read_gdf, parse_gdf, write_gdf and dump_gdf.

An old issue #455 exists for this but was closed at the time because the two main applications known to use this format - Gephi and GUESS - also support other formats that NetworkX can read/write. However, some of the research software we use (e.g. 4CAT) only supports GDF, and support for it in NetworkX would make it easier for people to use the output of that software, and would allow us to integrate NetworkX's functionality in our tools.

This is my first PR for NetworkX so please tell me if extra work would be needed before it can be merged!

@rossbar
Copy link
Contributor

rossbar commented Jul 22, 2021

From a quick search on the internet I'm inclined to agree with the conclusion in #455 - it might be more appropriate to add support for more commonly used network data formats to 4CAT than to add support for yet another not-particularly-common (AFAICT) text-based format to NetworkX. For example, it seems that TCAT, which appears to have inspired 4CAT, supports I/O in GEXF format.

@MridulS
Copy link
Member

MridulS commented Feb 10, 2023

I agree with @rossbar, thanks for the PR!! but I think we shouldn't be adding another text based graph format which seemingly (from a cursory internet search) doesn't seem to be supported in a lot of places. It heavily increases our maintenance burden.

And it looks like 4CAT has moved to GEXF digitalmethodsinitiative/4cat@176b15a ?

Anyway I'll go ahead and close this PR, thanks again for your contribution!

@MridulS MridulS closed this Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants