Color Y formats #12399
Color Y formats #12399
Conversation
| __m128i y0 = _mm_shuffle_epi8(s0, _mm_setr_epi8(1, 3, 5, 7, 9, 11, 13, 15, 0, 2, 4, 6, 8, 10, 12, 14)); | ||
| __m128i y1 = _mm_shuffle_epi8(s1, _mm_setr_epi8(0, 2, 4, 6, 8, 10, 12, 14, 1, 3, 5, 7, 9, 11, 13, 15)); | ||
| _mm_storeu_si128(&dst[i], _mm_alignr_epi8(y0, y1, 8)); | ||
| const __m128i vmask = _mm_set1_epi16( 0x00ff ); |
There was a problem hiding this comment.
Can you explain the issue?
We had color Y8 working a few month ago
There was a problem hiding this comment.
Oh now sure if SSE was enabled
There was a problem hiding this comment.
It wasn't working for me. I can show you.
I checked the non-SSE mode, and it works fine. It's only this specific code that doesn't work.
And I cannot say I understand the code that was there before. I understand this code I wrote.
There was a problem hiding this comment.
@SamerKhshiboun can you check if SSE is enabled on your NUC where you captured this recording?
This was a client request so I want to make sure we don't ruin it somehow (even on ROS)
There was a problem hiding this comment.
I verified and found it work on D500 because we use M420 to Y8.
Issue is in YUYV to Y8
| @@ -123,15 +123,6 @@ namespace librealsense | |||
| _ds_color_common->register_metadata(); | |||
| } | |||
|
|
|||
| void d500_color::register_processing_blocks() | |||
There was a problem hiding this comment.
Note that this will have to be implemented on -priv when merged there.
d500_colorprocessing-blocks now inregister_color_processing_blocks()