-
Notifications
You must be signed in to change notification settings - Fork 3
Add support for multiple runs #357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
So that we can see when the major version of a package changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for multiple simulation runs in the interactive mode, allowing users to save and compare different parameter configurations. It also includes code style improvements (replacing explicit !== undefined checks with truthy checks) and updates several dependencies including Plotly, mathjs, and various development tools.
Changes:
- Added interactive "Runs" feature with color customization, visibility toggles, and live updates control
- Refactored graph panel data structures to support multiple traces with individual colors
- Updated code style to use truthy checks instead of explicit undefined comparisons
Reviewed changes
Copilot reviewed 23 out of 25 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| SimulationExperimentView.vue | Added runs management UI, live updates settings, and data combination logic |
| GraphPanelWidget.vue | Refactored to support multiple traces with individual colors |
| GraphPanelWidgetPalette.ts | New color palette for consistent trace coloring |
| Multiple other files | Code style improvements (truthy checks) and dependency updates |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 27 out of 29 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
At least, for now since it's not currently being used.
So that we can show/hide it.
… of the additional runs.
…all the other runs.
…he window loses focus.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 7 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 7 out of 9 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fixes #299.