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

Adding start and stop transaction powermeter cmds to the JsDCSupplySimulator module #625

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

SebaLukas
Copy link
Contributor

@SebaLukas SebaLukas commented Apr 8, 2024

Describe your changes

Adding start and stop transaction powermeter cmds to the JsDCSupplySimulator module

Issue ticket number and link

#624

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

@barsnick
Copy link
Contributor

barsnick commented Apr 8, 2024

In commit message and PR description: "DCSupplySimulator" -> "JsDCSupplySimulator"

BTW, the former did not obsolete the latter because the latter also provides a powermeter interface, copying the supplied values as a meter reading. This could be added to the (C++) DCSupplySimulator if desired.

@SebaLukas SebaLukas changed the title Adding start and stop transaction powermeter cmds to the DCSupplySimulator module Adding start and stop transaction powermeter cmds to the JsDCSupplySimulator module Apr 8, 2024
@SebaLukas
Copy link
Contributor Author

Thank you for noticing. I changed the title and the description with the correct JsDCSupplySimulator module.

The cpp DCSupplySimulator module does not currently provide the powermeter interface. That is why I have not added the changes to the cpp module.

…lator module

Signed-off-by: Sebastian Lukas <[email protected]>
Signed-off-by: pietfried <[email protected]>
@SebaLukas SebaLukas force-pushed the fix/adding_transaction_to_dcpowersupplysim branch from 1a4b66c to 10c3157 Compare April 8, 2024 12:05
@SebaLukas SebaLukas merged commit 9ce24d5 into main Apr 8, 2024
5 checks passed
@SebaLukas SebaLukas deleted the fix/adding_transaction_to_dcpowersupplysim branch April 8, 2024 12:13
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.

3 participants