JuliaData / JuliaData/SplitApplyCombine.jl
Warning with groupreduce
Open
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 154
- Forks
- 16
- PR merge metrics
- No merged PRs in 30d
Description
When running the groupreduce example I get the following warning:
julia> groupreduce(iseven, +, 1:10)
┌ Warning: use values(kwargs) and keys(kwargs) instead of kwargs.data and kwargs.itr
│ caller = groupreduce(op::typeof(+), groups::MappedArray{Bool, 1, typeof(iseven), UnitRange{Int64}}, values::UnitRange{Int64}; kw::Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}}) at group.jl:125
└ @ SplitApplyCombine ~/.julia/packages/SplitApplyCombine/ulZAx/src/group.jl:125
2-element Dictionaries.Dictionary{Bool, Int64}
false │ 25
true │ 30
Contributor guide
No contributing guide indexed for this repository
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
Start in src/group.jl at line 125, which the warning identifies as the caller, and reproduce the issue with the reported groupreduce(iseven, +, 1:10) example. Check the current Julia API guidance for Base.Pairs access and confirm the warning disappears while the grouped totals remain unchanged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- data
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100