Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Improve IcebugMemGraph loading#33

Merged
adsharma merged 3 commits into
mainfrom
minor_fixes
May 28, 2026
Merged

Improve IcebugMemGraph loading#33
adsharma merged 3 commits into
mainfrom
minor_fixes

Conversation

@adsharma

Copy link
Copy Markdown
Contributor

Summary

  • Heterogeneous src/dest is now rejected.
  • Chunked Arrow columns now have an explicit combine path.
  • Missing duckdb/pytest/cython dependency issue is addressed in pyproject.toml.
  • Update README for directed IcebugMemGraph construction.

Validation

  • PYTHONPATH=../icebug-format uv run --no-sync python -m pytest networkit/test/test_icebug_mem_graph.py -q

@adsharma adsharma marked this pull request as ready for review May 27, 2026 23:03
@adsharma adsharma merged commit 9fdaa80 into main May 28, 2026
5 checks passed
@adsharma adsharma deleted the minor_fixes branch May 28, 2026 00:00
@aheev

aheev commented May 28, 2026

Copy link
Copy Markdown
Contributor

Heterogeneous src/dest is now rejected.

What's the motive?

@adsharma

Copy link
Copy Markdown
Contributor Author

If you have src = user, dest = city, icebug isn't setup to handle it. It's possible to make it work, but I worry that the code will further diverge from networkit. It's also not clear that the 100+ algorithms in the package are setup to deal with such graphs.

@adsharma

Copy link
Copy Markdown
Contributor Author

Gemini:

Unlike NetworkX, which has a dedicated bipartite module for tasks like projections (converting a user-item graph into a user-user graph), NetworKit lacks a broad suite of bipartite-specific convenience functions [8].

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