Picking "Grover's Algorithm" on Unstructured Search is ignored: the solver name maps to SAT's Grover solver
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- javascript, redux
- Domain
- frontend
Research direction
Start in SearchBarExtensible.js and trace how onChange maps the selected label using getKeyByValue and how the current problem’s solver list is provided. Reproduce the duplicate "Grover's Algorithm" labels for SAT and Unstructured Search, then constrain the lookup to that problem or use class names as option values. Done means selecting Grover’s Algorithm on Unstructured Search activates its Grover solver instead of leaving brute force selected.
Written by the indexing model from the issue text.
Description
In SearchBarExtensible.js, onChange maps the chosen label back to a class with getKeyByValue(optionsMap, label). The solver name map covers every problem, and both SATGroverSolver and Unstructured Search's Grover solver are named "Grover's Algorithm". So choosing it on Unstructured Search resolves to SATGroverSolver, which isn't in that problem's list, and the choice is silently ignored: the input shows "Grover's Algorithm" but the page keeps solving with the brute-force solver.
Fix idea: map labels to classes only within the current problem's solver list (or keep the class name as the option value), so equal display names on different problems can't collide.
Found while checking Redux#625 (quantum solver failures) in Redux_GUI, Oct 22.
- Dominant language
- JavaScript
- Stars
- 3
- Forks
- 18
- Avg merge
- 22h 57m
- Merged PRs (30d)
- 37
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- Ships a Dockerfile or Docker Compose file
- Has a pull request template
- No contributing 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.
More from ReduxISU/Redux_GUI
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
About Us: add an "In the News" section, starting with the Idaho State Journal BengalBot articleOpenenhancement
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ReduxISU/Redux_GUI#337 · 1 comment ·
Maintainers usually reply within 1 day
All issues in ReduxISU/Redux_GUI
Similar issues
-
bug require-triage
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
expressjs/expressjs.com#2549 · 1 reaction ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
vercel-labs/agent-skills#348 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
NanmiCoder/dsh-agent-teams#289 ·
Maintainers usually reply within 1 day