Skip to content

Conversation

@pmai
Copy link
Contributor

@pmai pmai commented Jan 17, 2018

This optional attribute allows sensor models that want to approximate
the influence of see-under without needing access to the whole vehicle
geometry. Addresses the part of issue #99 that can be addressed
directly in ground truth data.

@pmai pmai added the FeatureRequest Proposals which enhance the interface or add additional features. label Jan 17, 2018
@pmai pmai added this to the v2.3.0 milestone Jan 17, 2018
@pmai pmai self-assigned this Jan 17, 2018
@pmai pmai requested review from a user and hschoen January 17, 2018 21:14
Copy link
Contributor

@hschoen hschoen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good! This is exactly what I meant. One could take into account in the comment the case of an attached trailer or semi-trailer (second underbody plane): "Static minimal distance in [m] of underbody plane(s) to ground surface "

@pmai pmai force-pushed the feature/radar-specific-extensions branch from 332ffa0 to 6d4c6bf Compare January 19, 2018 22:21
This optional attribute allows sensor models that want to approximate
the influence of see-under without needing access to the whole vehicle
geometry.  Addresses the part of issue #99 that can be addressed
directly in ground truth data.
@pmai
Copy link
Contributor Author

pmai commented Jan 19, 2018

I think that in the case of attached trailers we have a second vehicle, with its own underbody plane?

@pmai pmai force-pushed the feature/radar-specific-extensions branch from 6d4c6bf to 082036c Compare January 19, 2018 22:35
@pmai pmai merged commit 082036c into master Jan 19, 2018
@pmai pmai deleted the feature/radar-specific-extensions branch March 27, 2018 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FeatureRequest Proposals which enhance the interface or add additional features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants