JuliaParallel/Dagger.jl

Make ArrayOp types non-concrete

Open

#193 opened on Jan 21, 2021

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Julia (706 stars) (86 forks)batch import
array interfacebughelp wanted

Description

There's no need, and iterative algorithms like NMF might build up a huge-but-regular type tree, slowing down the compiler.

Contributor guide

Tech stack
None
Domain
backendperformance
Issue type
performance
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
None
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
Look for the definition of ArrayOp types in the Dagger.jl source code (likely in src/array.jl or similar). The issue requires changing these types from concrete to non concrete to avoid building large type trees during iterative algorithms. After modifying the type definitions, run the existing test suite and consider testing with an NMF example to verify the performance improvement.