-
Notifications
You must be signed in to change notification settings - Fork 129
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
Fix media contents error in arrow format #986
Fix media contents error in arrow format #986
Conversation
160c614
to
347203b
Compare
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## releases/1.2.0 #986 +/- ##
==================================================
- Coverage 78.70% 78.67% -0.03%
==================================================
Files 231 231
Lines 26425 26421 -4
Branches 5260 5263 +3
==================================================
- Hits 20797 20787 -10
- Misses 4401 4408 +7
+ Partials 1227 1226 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
347203b
to
01f603e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you change the target branch of this PR to releases/1.2.0
?
b0607c8
to
2002928
Compare
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Summary
Image
media from bytes not a decoded numpy arrayPointCloud
whensave_media=False
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.
Signed-off-by: Inhyuk Andy Cho andy.inhyuk.jo@intel.com