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

unpack_yuy2 cuda mode #3623

Closed
wonshik opened this issue Mar 29, 2019 · 3 comments
Closed

unpack_yuy2 cuda mode #3623

wonshik opened this issue Mar 29, 2019 · 3 comments
Labels

Comments

@wonshik
Copy link

wonshik commented Mar 29, 2019

  • Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

  • All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)


Required Info
Camera Model { D435 }
Firmware Version (05.10.13.00)
Operating System & Version Linux (Ubuntu 16)
Kernel Version (Linux Only) 4.4.38-tegra)
Platform NVIDIA Jetson TX2
SDK Version { 2.17.1 }
Language C/C++
Segment {Robot}

Issue Description

<Describe your issue / question / feature request / etc..>

Even though I enable the BUILD_WITH_CUDA, I can see unpack_yuy2 process in profiler.
So, I check

image.cpp +345

There is #ifdef for cuda acceleration. but the problem is that CPU unpack code execute again after cuda calculation.
I think CPU unpack code should be blocked when BUILD_WITH_CUDA is enabled.

Please check this issue.
Thanks

@dorodnic dorodnic added the cuda label Mar 29, 2019
@dorodnic
Copy link
Contributor

Hi @wonshik
It is not entirely unlikely that this was broken in one of the merges. At the moment, we do not yet officially support or validate RealSense on Jetson (though working in that direction).
I'll look into it.
If you have a patch for this issue, we accept and encourage community contribution

@dorodnic
Copy link
Contributor

dorodnic commented Apr 1, 2019

Confirmed, pull-request submitted.
Thank you for pointing this out.

@dorodnic
Copy link
Contributor

dorodnic commented Apr 7, 2019

Should be resolved in 2.20

@dorodnic dorodnic closed this as completed Apr 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants