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

Updated depth colorization logic #809

Merged
merged 3 commits into from
Apr 24, 2023
Merged

Updated depth colorization logic #809

merged 3 commits into from
Apr 24, 2023

Conversation

Erol444
Copy link
Member

@Erol444 Erol444 commented Apr 13, 2023

Before / After
image

Another example, spatial_calculator_multi_roi.py
image

@Erol444 Erol444 marked this pull request as ready for review April 13, 2023 16:29
@Erol444 Erol444 changed the title Started updating depth colorization logic Updated depth colorization logic Apr 13, 2023
@Erol444
Copy link
Member Author

Erol444 commented Apr 24, 2023

Processing time, code here: https://pastebin.com/HgjsdTwK. For depthai-python examples I'd use proposed approach, as depth-to-disp takes 30LoC. In SDK we will rather use depth-to-disp. @szabi-luxonis

400P:
depth to disp avrg: 2.5ms
depth percentile (my method) avrg: 4 ms
normalize (current method): 1ms
800P:
percentile 15ms
normalize: 2.5ms
depth to disp: 10ms

@Erol444 Erol444 merged commit e6ac495 into main Apr 24, 2023
@Erol444 Erol444 deleted the update_depth_visualziation branch April 24, 2023 10:06
@Erol444 Erol444 mentioned this pull request Apr 24, 2023
3 tasks
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

Successfully merging this pull request may close these issues.

2 participants