dwavesystems / dwavesystems/dwave-optimization

Consider dropping `DynamicArrayTestingNode`

Open
#593 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
31
Forks
36
Avg merge
16h 55m
Merged PRs (30d)
8

Description

It was originally written before we had more powerful reshaping/resizing methods. It's now much easier to use other decision types with `BroadcastToNode`.

Contributor guide

Open the contributing guide

Research direction

Start by locating DynamicArrayTestingNode and all of its usages, then read the surrounding reshaping/resizing methods and BroadcastToNode implementation. Determine whether the node can be removed without breaking supported decision types, update affected references and tests, and verify the test suite passes with no remaining uses.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.