Skip to content

Updated Reduction Tooltips, Problems Filter, Etc... - #161

Merged
wrigjl merged 1 commit into
ReduxAPI_GUIfrom
feature/tag-system-expansion-375-379
Aug 13, 2026
Merged

Updated Reduction Tooltips, Problems Filter, Etc...#161
wrigjl merged 1 commit into
ReduxAPI_GUIfrom
feature/tag-system-expansion-375-379

Conversation

@Trosper3

Copy link
Copy Markdown
Contributor

Frontend — Redux_GUI repo, branch feature/tag-system-expansion-375-379

One commit, b423a3e — Updated Reduction Tooltips, Problems Filter, Etc..., on top of 35f42bc:

  • #375 (filter excludes unchecked): ProblemFilterMenu's active-filter count now sums every wired facet, not just two of four — previously the badge and "Clear filters" button ignored selections in unwired facets
  • #377 (tooltip styling): removed the grey background boxes from every tooltip section except the header, so all text entries match
  • #378 (simplify visualization types): visualization type now displays as its simplified category ("Graph" instead of "GraphD3"/"GraphLaTeX") via new visualizationCategories.js
  • #379 (filter by visualization type): new "Visualization type" facet in ProblemFilterMenu, grouped by category (Boolean/Circuit/Graph/Schedule/Set/Table); buildGroupedFacetOptions added to facetOptions.js
  • Reduction tooltip enhancement (your ask this session): ReduceToRowReact.js now shows Reduction type, Complexity bucket, and Big-O alongside the existing Reduction cost, mirroring the solver tooltip's pattern
  • Real bug fix: requestReductionInfo in components/redux/index.js was hitting a nonexistent route (${className}/info instead of ProblemProvider/info?interface=${className}), 404ing silently on every load and leaving every reduction tooltip field on its fallback text regardless of backend data. Fixed to delegate to requestInfo like every other tooltip does.

Linked to ReduxISU/Redux#396

@github-actions

Copy link
Copy Markdown

Redux Build System — CI Report

npm · commit d2fc634

Operation Status Summary Time
audit 5 high 1.1s
format-check 58 format · 31 import order 0.5s
lint 26 errors, 41 warnings 7.2s
typecheck ⏭️ no tsconfig.json
unit-test ⏭️ no test script in package.json
build built local/redux_gui:ci · 248MB 57.1s
integration-test 1 failed · 3 passed 20.7s
push not run — integration-test failed

Overall: ❌ 1 passed · 4 failed · 2 skipped · 1 blocked

❌ audit — 5 high
Severity Location Rule Message
high nanoid@<=3.3.16 GHSA-28wg-ghj8-5hjv nanoid: non-secure generators can loop indefinitely with negative size
high next@9.3.4-canary.0 - 16.3.0-preview.10 vulnerable dependency next
high postcss@<=8.5.22 GHSA-qx2v-qp2m-jg93 PostCSS has XSS via Unescaped </style> in its CSS Stringify Output
high sharp@<0.35.0 GHSA-f88m-g3jw-g9cj sharp inherited vulnerabilities in libvips: CVE-2026-33327, CVE-2026-33328, CVE-2026-35590, CVE-2026-35591
high svgo@3.0.0 - 3.3.3 GHSA-2p49-hgcm-8545 SVGO removeScripts plugin leaves some executable scripts intact
❌ format-check — 58 format · 31 import order
Severity Location Rule Message
error Tools/Constants.js format needs formatting
error Tools/ProblemInstanceParser.js format needs formatting
error biome.json format needs formatting
error components/ContributorCard.js format needs formatting
error components/Quantum/QuantumCircuitVisualizer.js assist/source/organizeImports Sort these imports.
error components/Quantum/QuantumCircuitVisualizer.js format needs formatting
error components/Quantum/circuitUtils.js format needs formatting
error components/Visualization/Graphvisualization.js format needs formatting
error components/Visualization/GraphvizWrapper.js assist/source/organizeImports Sort these imports.
error components/Visualization/QuantumCircuitVis.js assist/source/organizeImports Sort these imports.
error components/Visualization/QuantumCircuitVis.js format needs formatting
error components/Visualization/ReducedVisualization.js format needs formatting
error components/Visualization/constants/VisColors.js format needs formatting
error components/Visualization/constants/VisColorsArray.js assist/source/organizeImports Sort the exported names.
error components/Visualization/constants/VisColorsArray.js format needs formatting
error components/Visualization/openqasmToQText.js format needs formatting
error components/Visualization/svgs/DFATableSvgReact.js format needs formatting
error components/Visualization/svgs/DynamicTableSvgReact.js format needs formatting
error components/Visualization/svgs/LaTeXGraphSvgReact.js format needs formatting
error components/Visualization/svgs/NFATableSvgReact.js format needs formatting
… and 70 more
❌ lint — 26 errors, 41 warnings
Severity Location Rule Message
error components/Visualization/QuantumCircuitVis.js:152 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Problem.js:51 no-undef 'requestInfo' is not defined.
error components/hooks/ProblemProvider/Problem.js:73 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:108 react-hooks/immutability Error: Cannot access variable before it is declared
error components/hooks/ProblemProvider/Reducer.js:132 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:152 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:171 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:190 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:211 react-hooks/immutability Error: Cannot access variable before it is declared
error components/hooks/ProblemProvider/Reducer.js:215 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:51 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Reducer.js:74 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Solver.js:138 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Solver.js:24 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/hooks/ProblemProvider/Verifier.js:87 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/pageblocks/ProblemRowReact.js:152 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/pageblocks/ProblemRowReact.js:161 react-hooks/immutability Error: This value cannot be modified
error components/pageblocks/VerifyRowReact.js:43 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/pageblocks/VerifyRowReact.js:49 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
error components/pageblocks/VisualizeRowReact.js:133 react-hooks/set-state-in-effect Error: Calling setState synchronously within an effect can trigger cascading renders
… and 47 more

@wrigjl wrigjl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LOve it!

@wrigjl
wrigjl merged commit 3989e98 into ReduxAPI_GUI Aug 13, 2026
7 checks passed
@wrigjl
wrigjl deleted the feature/tag-system-expansion-375-379 branch August 13, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants