JuliaArrays / JuliaArrays/BlockArrays.jl
Ambiguity causing broadcast overload
Open
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 254
- Forks
- 39
- PR merge metrics
- No merged PRs in 30d
Description
Found with:
Expression: isempty(Test.detect_ambiguities(OrdinaryDiffEq))
Evaluated: isempty(Tuple{Method, Method}[(materialize!(dest, bc::Base.Broadcast.Broadcasted{BS, Axes, F, Args} where {Axes, F, Args<:Tuple}) where BS<:BlockArrays.AbstractBlockStyle in BlockArrays at /home/runner/.julia/packages/BlockArrays/5iQPM/src/blockbroadcast.jl:147, materialize!(af::OrdinaryDiffEq.ArrayFuse, src::Base.Broadcast.Broadcasted) in OrdinaryDiffEq at /home/runner/work/RecursiveArrayTools.jl/RecursiveArrayTools.jl/downstream/src/wrappers.jl:31)])
Aren't you supposed to be strictly typing dest if you overload materialize!?
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 with the reported Test.detect_ambiguities(OrdinaryDiffEq) expression and inspect the materialize! methods shown in BlockArrays/src/blockbroadcast.jl and RecursiveArrayTools/downstream/src/wrappers.jl. Determine why the overloads are ambiguous and confirm that the ambiguity check no longer reports this method pair when the issue is resolved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- data
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100