Documentation page for packages that extend Graphs.jl #500
Documentation page for packages that extend Graphs.jl #500Krastanov merged 9 commits intoJuliaGraphs:masterfrom
Conversation
Benchmark Results (Julia v1)Time benchmarks
Memory benchmarks
|
|
This looks pretty great to me. I would suggest add IGraphs.jl and NautyGraphs.jl to the docs/Project.toml and then adding a subsection "Dispatching to algorithm implementations in wrapper packages" with a small example and then an automated listing using |
|
I tried something in that direction. Let me know if that works. |
Krastanov
left a comment
There was a problem hiding this comment.
This looks good to me! Could you please change the using statements to imports instead?
…` in docs/graphalgorithms.md
|
I am a bit puzzled by the CI failures. Seems like adding NautyGraphs/IGraphs to test/Project.toml is necessary to make doctests pass. I am still a bit confused about the failure to build the docs. On my machine it works without issues. |
|
The tests also partially build the docs so that was probably why you need them there too. Do you happen to use a different version of Julia locally? That particular function which is failing seems to depend on unstable julia internals. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #500 +/- ##
=======================================
Coverage 97.29% 97.29%
=======================================
Files 126 126
Lines 7662 7662
=======================================
Hits 7455 7455
Misses 207 207 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
if something does not work on specific OSs or old versions of julia, we can work around it -- no need to worry about it for now |
|
A potential fix for the method enumerator (in situations where you have more than one type parameters in your parametric type): |
|
Doctest pass and docs build normally now. The only issue is that, because of |
Run JuliaFormatter on docs/make.jl to fix code-style CI. Skip doctests on Windows since NautyGraphs is not available there. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
It was suggested by @Krastanov to include a documentation page mentioning external packages that extend the functionality of Graphs.jl. While this already exists for packages that implement custom graph types, this PR adds a documentation page for packages that implement additional algorithms (GraphsColoring, GraphsMatching, ...), or wrap external graph libraries in a compatible way (Igraphs, NautyGraphs, ...).
I am not sure how to best add these packages to the documentation, but I hope that my initial attempt here can serve as a starting point for discussion. The text describing the various packages in particular should be seen as placeholder text -- ideally, the package descriptions would be rewritten/improved by the respective package authors.