-
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
Implement DatumaroBinaryFormat to export/import the image dataset completely #829
Implement DatumaroBinaryFormat to export/import the image dataset completely #829
Conversation
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
- Refactor CommonSemanticSegmentation unit tests as well Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
- Support DatasetItem by Datumaro binary format but annotations are not yet. Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
d89710d
to
dbb7e6b
Compare
…maro-binary-format-dataset-item Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
dbb7e6b
to
aa6b1bd
Compare
…tation-mapper Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #829 +/- ##
===========================================
+ Coverage 78.47% 78.48% +0.01%
===========================================
Files 191 196 +5
Lines 23911 24304 +393
Branches 4912 4946 +34
===========================================
+ Hits 18765 19076 +311
- Misses 4044 4127 +83
+ Partials 1102 1101 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
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 at Codecov. |
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.
It seems good for me, except Mapper
classes. I understood the concept of it, but I could not catch up deeply for Mapper
classes, so this should be covered through others reviews.
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.
Please hold this when finalizing the discussion about encryption.
Summary
Image
media for now).How to test
Checklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.