Skip to content
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

IMDSimulator: Add C++ simulation module #498

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

barsnick
Copy link
Contributor

This C++ module supplements JsIMDSimulator.

Co-authored-by: Cornelius Claussen [email protected]
Co-authored-by: Fabian Hartung [email protected]
Co-authored-by: Moritz Barsnick [email protected]

@corneliusclaussen
Copy link
Contributor

Some of the comments from #499 apply here as well

description: Measurement update interval in milliseconds
type: integer
default: 1000
enable_external_mqtt: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Since mqtt is not used directly in the module, this can be set to false.

@barsnick barsnick force-pushed the cb-add-imd-simulator branch 2 times, most recently from c8609c2 to d81a8df Compare January 22, 2024 16:34
Copy link
Contributor

@corneliusclaussen corneliusclaussen left a comment

Choose a reason for hiding this comment

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

Please squash into one commit and adapt commit message before merging

This C++ module replaces JsIMDSimulator.

Co-authored-by: Cornelius Claussen <[email protected]>
Co-authored-by: Fabian Hartung <[email protected]>
Co-authored-by: Moritz Barsnick <[email protected]>

Signed-off-by: Fabian Hartung <[email protected]>
Signed-off-by: Moritz Barsnick <[email protected]>
@barsnick barsnick force-pushed the cb-add-imd-simulator branch from d81a8df to 84f799d Compare January 24, 2024 14:30
@barsnick barsnick merged commit e1bfa0a into EVerest:main Jan 24, 2024
3 of 4 checks passed
@barsnick barsnick deleted the cb-add-imd-simulator branch January 24, 2024 14:46
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.

4 participants