-
Notifications
You must be signed in to change notification settings - Fork 130
fix(docs): corrected vector from rear axis to sensor mounting point #456
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
fix(docs): corrected vector from rear axis to sensor mounting point #456
Conversation
Signed-off-by: Katrin Mehl <katrin.mehl@parson-europe.com>
|
update of svg after input from ccb. Please have a look if this fits now: A description about bbcenter_to_rear is documented here Which I used for naming the end of the vector from center of the bounding box to the rear axle. |
Signed-off-by: Katrin Mehl <katrin.mehl@parson-europe.com>
Signed-off-by: Katrin Mehl <katrin.mehl@parson-europe.com>
Signed-off-by: Katrin Mehl <katrin.mehl@parson-europe.com>
stefancyliax
left a comment
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.
Image was approved in the SensorModelling WG meeting 29.10.2021


Reference to a related issue in the repository
#447
Add a description
Changed the start of vector pointing to the physical mounting position of the sensor.
It was: Center of the bounding box.
It is: Middle of the rear axis (coordinate system).
It is a bug fix according to the related issue.
It shall enhance the understanding of MovingObjects and the related coordinate systems.
It has not been tested. But since this is only an svg not code-related, no testing is needed.
Hence, I don't think it will break any existing functionality or force me to update to a new version.
Mention a member
@HendrikAmelunxen please reviewing proposed changes.
Check the checklist