Skip to content

Add two-output graph for visualization.py. Add csv validation for dashboard. - #52

Merged
tupui merged 22 commits into
Simulation-Decomposition:mainfrom
justushelo:45-two-output-graph
Apr 27, 2026
Merged

Add two-output graph for visualization.py. Add csv validation for dashboard.#52
tupui merged 22 commits into
Simulation-Decomposition:mainfrom
justushelo:45-two-output-graph

Conversation

@justushelo

@justushelojustushelo commented Mar 9, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor
  • Add two-output option for visualization.py similar to Matlab implementation.
  • Add csv validation to dashboard to catch incorrect delimiters or incorrect column names.
  • Add two-output into simdec_app.py
  • Ship from panel to wasm (Netlify build successfull).
  • ClosesTwo-output graph call #45. Closeserror messages #50. ClosesClient side only #53.

tupui
tupui requested changes Mar 9, 2026
View reviewed changes

@tupuitupui left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! I left some comments.

Comment threadsrc/simdec/visualization.py Outdated
Comment threadsrc/simdec/visualization.py Outdated
Comment threadsrc/simdec/visualization.py Outdated
Comment threadpanel/simdec_app.py Outdated
@justushelo
Add two-output visualization and robust CSV parsing
d90dd4d
Created a new function for two-output visualization. Added a
try/except block for CSV parsing in the dashboard. If a wrong
delimiter or invalid character is detected, it loads stress.csv
to stop buffering and prevent reactive cascade crashes.
ClosesSimulation-Decomposition#45ClosesSimulation-Decomposition#50
@justushelo
Fix column name regex validation
a9af109
@justushelo
feat: Client-side deployment, two-output graph, csv-validation
8718ee6
* Static client-side WASM-deployment (ClosesSimulation-Decomposition#53)
- Replaced Python/Jinja backend templates with fully static HTML.
- Included light/dark theme toggle.
- Added netlify.toml for Netlify build.
- Added package.json for Netlify build triggers.
- Added scripts/build_wasm.sh for installing dependencies,
building SimDec wheel, converting Panel to Pyodide,
and copying necessary files.
- Needed to add data/stress.csv to panel/ for conversion to WASM.
* Fixed error handling with faulty CSV files (ClosesSimulation-Decomposition#50)
- Faulty files display error message
- Faulty files keep buffering icon in UI
* Two-output graph (ClosesSimulation-Decomposition#45)
- Two-output graph is in a new function as in previous PR.
@justushelo
build_wasm.sh update
d4c90f9
Updated the build_wasm.sh script to work with netlify
@justushelo
Update build script command to use bash
decc2ff
@justushelo
Guard rails for build_wasm.sh
d0f011e
Checking file existence.
@justushelo
Update package.json
3c3f762
Updated to "serve": "python3..."
@justushelo
Update paths for Pyodide worker conversion
2aa9e96
@justushelo
Enable logging in build_wasm.sh
e01b7a3
Added logging to the build_wasm.sh script for better debugging.
@justushelo
Refactor panel convert command in build_wasm.sh
29a7d12
Updated the panel convert command to use a relative path for requirements.
tupui
tupui approved these changes Apr 8, 2026
View reviewed changes

@tupuitupui left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor thing, happy to merge otherwise!

Comment threadsrc/simdec/visualization.py
Comment threadpanel/simdec_app.py
tupui
tupui approved these changes Apr 27, 2026
View reviewed changes

@tupuitupui left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good job 👏 let's try all of that

@tupui
tupui merged commit b0727ef into Simulation-Decomposition:mainApr 27, 2026
4 checks passed
@justushelo
justushelo deleted the 45-two-output-graph branch April 29, 2026 09:57
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tupuitupuitupui approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Client side only error messages Two-output graph call

2 participants

@justushelo@tupui