-
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
Refactor Image/PointCloud 2 #944
Refactor Image/PointCloud 2 #944
Conversation
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>
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>
@@ -165,7 +165,7 @@ def _load_inference_from_subset(extractor, subset_name): | |||
width, height = item.media.size | |||
data_df["Width"].append(width) | |||
data_df["Height"].append(height) | |||
data_df["ImagePath"].append(item.media.path) | |||
data_df["ImagePath"].append(getattr(item.media, "path", None)) |
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.
I guess we would rather add path
property which returns None
to the base media class.
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.
Yes, indeed. path
is way more used than I thought across Datumaro. Lets make it happen in another PR.
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
Signed-off-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com>
3e84aeb
to
c9ea1c3
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## releases/1.2.0 #944 +/- ##
===============================================
Coverage 78.68% 78.69%
===============================================
Files 231 231
Lines 26408 26417 +9
Branches 5257 5258 +1
===============================================
+ Hits 20780 20789 +9
+ Misses 4403 4401 -2
- Partials 1225 1227 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
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
A following work of #911, #921
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.