Add a new custom message for publishing world stats. #409
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also cleans up the protobuf descriptors generator (cmake) and makes sure the .proto files are installed.
stonier
approved these changes
May 15, 2018
|
@caguero I factored out the hardcoded weight value in a constant and adjusted it so it remains a bit more stable on the UI while still catching up with realtime changes (you can see it in action by running the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Requires pull request # 102 from Delphyne Gui.
Generates a custom version of
WorldStatisticsmessage until we can use the updated message in Ignition Msgs. It also populates thereal_time_factorfield.Note: this PR is built atop pull request #407 , which should be merged first.