Skip to content

Fix simulator timestamping, add version+hostname to model_info - #50

Merged
JJL772 merged 2 commits into
mainfrom
pr-misc-changes
Aug 12, 2026
Merged

Fix simulator timestamping, add version+hostname to model_info#50
JJL772 merged 2 commits into
mainfrom
pr-misc-changes

Conversation

@JJL772

@JJL772 JJL772 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

What it says on the tin. Fix timestamping in the simulator (only needed for example code), add hostname+lume pva version to the model_info pv

@JJL772
JJL772 requested a review from tangkong August 11, 2026 23:33

@tangkong tangkong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

Comment thread lume_pva/runner.py Outdated
Comment thread lume_pva/runner.py
@JJL772
JJL772 merged commit ffd974c into main Aug 12, 2026
13 of 14 checks passed
@JJL772
JJL772 deleted the pr-misc-changes branch August 12, 2026 16:28
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