Skip to content
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

Units - rtbCamerasPlace #35

Open
wandell opened this issue Aug 15, 2017 · 0 comments
Open

Units - rtbCamerasPlace #35

wandell opened this issue Aug 15, 2017 · 0 comments
Assignees

Comments

@wandell
Copy link
Contributor

wandell commented Aug 15, 2017

In a number of places, and rtbCamerasPlace is one example, there are factors that scale by 1000 to deal with meters and millimeters. There are rarely comments about this near the code. We need to catch these locations and deal with units more explicitly.

This might all be relatively new with code from HB and me. If we don't deal with the units throughout, say distance is sometimes meters and sometimes millimeters, we will have long term problems.

@wandell wandell self-assigned this Aug 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant