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
tm_sensor - doxygen doc of relocalization functions #4685
Conversation
bool import_relocalization_map(const std::vector<uint8_t>& lmap_buf) const override; | ||
|
||
/** | ||
* Creates a named virtual landmark in the current map, known as static node. |
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.
* Creates a named virtual landmark in the current map, known as static node. | |
* Creates a named virtual landmark in the current map, known as a static node. |
src/tm2/tm-device.h
Outdated
/** | ||
* Creates a named virtual landmark in the current map, known as static node. | ||
* The static node's pose must be giving relative to the origin of coordinates of T265 poses. | ||
* This function fails if the current tracker confidence is not high. |
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.
"not high" means what, below 3? Should we be specific?
src/tm2/tm-device.h
Outdated
* Creates a named virtual landmark in the current map, known as static node. | ||
* The static node's pose must be giving relative to the origin of coordinates of T265 poses. | ||
* This function fails if the current tracker confidence is not high. | ||
* \param[in] guid unique name of the static node. If a static node with the same name alrady exists |
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.
typo: alrady
|
||
/** | ||
* Gets the current pose of a static node that was created in the current map or in an imported map. | ||
* Static nodes of imported maps are available after relocalizing the imported map. |
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.
* Static nodes of imported maps are available after relocalizing the imported map. | |
* Static nodes of imported maps are available after a relocalization to the imported map has occurred. |
And as a bonus, how would anyone know that a relocalization has occurred? sounds like that currently is non-trivial, so it may be best to not comment on it it until it's easy for user to catch those notifications.
Co-Authored-By: Michael Harville <harville@stanfordalumni.org>
No description provided.