JuliaGraphs / JuliaGraphs/Graphs.jl
Deprecated CycleGraph usage needs removing.
Open
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 538
- Forks
- 128
- Avg merge
- 10h 25m
- Merged PRs (30d)
- 2
Description
Rogue instance of CycleGraph in test/louvain.jl:
Warning: `CycleGraph(n)` is deprecated, use `cycle_graph(n)` instead.
│ caller = macro expansion at louvain.jl:72 [inlined]
└ @ Core ~/work/Graphs.jl/Graphs.jl/test/community/louvain.jl:72
Happy to take.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open test/community/louvain.jl and inspect the CycleGraph usage reported at line 72. Run the Louvain/community test first, then verify the deprecated warning is gone and the test still passes after the usage is updated.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100