Add two-output graph for visualization.py. Add csv validation for dashboard.#52
Open
justushelo wants to merge 12 commits intoSimulation-Decomposition:mainfrom
Open
Add two-output graph for visualization.py. Add csv validation for dashboard.#52justushelo wants to merge 12 commits intoSimulation-Decomposition:mainfrom
justushelo wants to merge 12 commits intoSimulation-Decomposition:mainfrom
Conversation
…hboard to catch wrong delimiters or incorrect column names. Closes Simulation-Decomposition#45. Closes Simulation-Decomposition#50.
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. Closes Simulation-Decomposition#45 Closes Simulation-Decomposition#50
* Static client-side WASM-deployment (Closes Simulation-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 (Closes Simulation-Decomposition#50) - Faulty files display error message - Faulty files keep buffering icon in UI * Two-output graph (Closes Simulation-Decomposition#45) - Two-output graph is in a new function as in previous PR.
Updated the build_wasm.sh script to work with netlify
Checking file existence.
Updated to "serve": "python3..."
Added logging to the build_wasm.sh script for better debugging.
Updated the panel convert command to use a relative path for requirements.
tupui
approved these changes
Apr 8, 2026
Member
tupui
left a comment
tupui
left a comment
There was a problem hiding this comment.
Minor thing, happy to merge otherwise!
Member
There was a problem hiding this comment.
Ok, should we also use your new function for visualising 2 outputs?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add two-output option for visualization.py similar to Matlab implementation. Add csv validation to dashboard to catch incorrect delimiters or incorrect column names. Closes #45. Closes #50.