-
Notifications
You must be signed in to change notification settings - Fork 628
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
New variance field for the Range message #1270
Comments
@jlblancoc awesome! I saw the commit, this a clever way of making sure the member exist. |
Exactly 👍 |
However, for some reason, the https://build.ros2.org/job/Hdev__mrpt2__ubuntu_jammy_amd64/349/console
I'll investigate it. |
This was added on March 15. It didn't make Humble. |
Maybe try it with declval?
|
Yeap, exactly! :-) |
@jolting It was something else... I just found (the hard way..) that everything inside an PS: before that, I also tried the declval trick without luck... :-( Thanks!! |
The
Range
message will get an additional field for the variance.I don't know if this is relevant for you guys, but this issue is simply a notification, feel free to close it if it's not a change that you could benefit from.
You can see details about the PR here
The text was updated successfully, but these errors were encountered: