Skip to content

Add the unit to the title of the simulation data we broadcast#105

Merged
agarny merged 3 commits intoABI-Software:mainfrom
agarny:simulation-data
Mar 14, 2026
Merged

Add the unit to the title of the simulation data we broadcast#105
agarny merged 3 commits intoABI-Software:mainfrom
agarny:simulation-data

Conversation

@agarny
Copy link
Copy Markdown
Collaborator

@agarny agarny commented Mar 14, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 14, 2026 05:22
@agarny agarny merged commit b54008b into ABI-Software:main Mar 14, 2026
3 checks passed
@agarny agarny deleted the simulation-data branch March 14, 2026 05:23
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the simulation data payload emitted by SimulationVuer to include units in the series title, aligning the broadcast metadata with the newer OpenCOR data shape (and bumps the package/dependency versions accordingly).

Changes:

  • Read per-series simulation data once (simData) and use its data field as the y values.
  • Append the series unit to the emitted data.title.
  • Bump package version and update @opencor/opencor dependency (and lockfile) to the corresponding release.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/components/SimulationVuer.vue Uses the new simulation data structure and includes units in emitted series titles.
package.json Bumps package patch version and updates @opencor/opencor dependency range.
package-lock.json Locks the updated package/dependency versions and integrity/resolution for @opencor/opencor.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

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