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

Added Interface for Netzschleuder .gml.zst files #818

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

AdityaChaubeyIITM
Copy link

@AdityaChaubeyIITM AdityaChaubeyIITM commented Mar 22, 2025

I have added a new file repositories.py under src/igraph/io/ , for the creation of graph from netzschleuder.
It lets the user just specify the name of the dataset and get the single network in the dataset as a result if there is only one network in the dataset, otherwise it asks for the sub-network's name as well.
It temporarily downloads the .gml.zst file, decompresses it and then creates a graph out of the data. The graphs scope is limited to the data presentation in .gml files in netzschleuder.
This is supposed to be an attempt to open discussion on methods to import netzschleuder dataset via '.gml' datatype files. It can be further modified to take care of the issues arising due to incorrect handling of edge attributes in .gml files.

  • By submitting this pull request, I assign the copyright of my contribution to The igraph development team.

Changed the API call to "urllib" instead of "requests" to avoid import errors in the build
@szhorvat
Copy link
Member

szhorvat commented Mar 22, 2025

Just a heads up that commit b458a31 on the main branch broke some tests. You don't need to worry about these, I'll look into sorting them out.

@AdityaChaubeyIITM
Copy link
Author

image
thank you @szhorvat !
However for this particular case Build test says it broke due to missing zstandard module, I guess that can be fixed by adding zstandard to the requirements.txt.
Please let me know if that is what is needed, I shall correct it from my end then.

Similar issue happened earlier with the previous one adaeae0 also broke because of missing requests.

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

Successfully merging this pull request may close these issues.

2 participants