Enhance roi depth mapping - #85
Conversation
|
So trying this on Mac and running into problems: |
|
And then the following command doesn't overlay the bounding box on the left stream: |
|
OK, using And this seems to work well: As discussed internally I'm wondering if it would make sense to allow an option for the neural networks to get the full field of view of the color camera (to 'squish' the image into the square aspect ratio of the say 300x300 pixel model), such that the whole field of view for color camera is still available even when square models are used. |
Luxonis-Brian
left a comment
There was a problem hiding this comment.
I just tested with BW1097 (CM3+) as well, and the bounding boxes on the depth_sipp look good. I did notice the offset in the box locations we had been discussing, but it's not as bad as I was thinking. One thing I did notice was that the previewout and depth_sipp (-bb) streams would display slightly different location values at each frame. I noticed that one was not following the other's values, that they each seemed to be using different values. I figure this is just a normal artifact of the metaout running at 30fps in the background and the two streams grabbing the values at slightly different times. I don't think it's a problem, it's just something I happened to notice after I took a picture of my computer monitor.
LGTM.
(was missed from previous commits)
Previously was hardcoded to 5m.
|
I missed overlaying the BB on left/right streams. It's fixed now. |
|
Thanks. LGTM @alex-luxonis and since this isn't a regression but something already on Master I think we're clear to merge. Thanks again! |
|
Sweet thanks! |
Enhance roi depth mapping



left/right/depth_sipp/depth_color_hstreams. Issue Add Option to Overlay Bounding Box Ondepth_sipp#75previewoutwith some NN models. Issue Strange artifacts showing on right side of preview image #72