-
Notifications
You must be signed in to change notification settings - Fork 243
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 camera frustum marker #181
Comments
If you want to avoid adding the dependency on |
Yeah, I really just need the two focal lengths and the image height and width, so I could just make each of those values an argument. I'd prefer using the |
@JStech rviz_visual_tools already depending on |
Ah, so it does--I didn't look beyond the main source file, I admit. I'll go ahead and put together a PR for this. |
Cool feature, was this ever implemented or any plans to? |
I propose adding a marker type that draws a camera frustum given the optical frame, camera intrinsics (
sensor_msgs::CameraInfo
message), and scale. The marker would look like this:And the function signature would be something like,
There's already a function to do this in MoveIt Calibration, and I'd basically copy it here (and then use the version here in MoveIt Calibration). I recently ran into another situation where I'd like to draw a frustum, and so I thought it'd be a good addition to this package. Before writing it, though, I thought I'd double-check that people would want that included,
since it would add the dependency on.sensor_msgs
The text was updated successfully, but these errors were encountered: