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
Add Single horizon measurement #4729
Add Single horizon measurement #4729
Conversation
2c75489
to
03666b0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, please rebase and squash in the one requested change.
|
||
namespace control_system::measurements { | ||
template <::domain::ObjectLabel Horizon> | ||
struct SingleHorizon : tt::ConformsTo<protocols::Measurement> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add some dox just so it shows up?
namespace control_system::measurements { | ||
template <::domain::ObjectLabel Horizon> | ||
struct SingleHorizon : tt::ConformsTo<protocols::Measurement> { | ||
struct SubMeasurement : tt::ConformsTo<protocols::Submeasurement> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[optional] Your capitalization here is kind of weird.
03666b0
to
3a2f915
Compare
Squashed both of the changes and also added some docs to the |
Proposed changes
And use it in the shape control system. This is similar to the BothHorizons measurement, except it is only for one horizon.
Upgrade instructions
Code review checklist
make doc
to generate the documentation locally intoBUILD_DIR/docs/html
.Then open
index.html
.code review guide.
bugfix
ornew feature
if appropriate.Further comments
Depends on and includes #4728.