-
Notifications
You must be signed in to change notification settings - Fork 298
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
Add plot method for VHR10 dataset #847
Conversation
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.
Can you add example plots?
ae05b72
to
43a7162
Compare
2febe32
to
0ad37dd
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.
More minor comments. My main concerns so far are:
- Handling of COCO polygons seems to be much more complex than it used to be
- Lots of very recent dependency versions
It would be good to have a justification for these things. Complexity is fine if it's necessary but I always try to avoid it if it isn't. I haven't gotten a chance to play around with finding the minimum versions, but you can always drop the version repeatedly in the file until CI starts to fail to find a lower bound.
14582c6
to
7251032
Compare
d7cde67
to
7ba58cd
Compare
3c04942
to
6eb0afd
Compare
Will move |
* Add plot fn & rename nwpu -> vhr10 * Update test data & add data.py * Add scikit-image dependency * Move scikit-image dep to extras * Comment out check for coverage * Move skimage import * Update requirements * Test requirements * Update requirements * Update * Update * Update * Update * Update * Update min scipy * Update min scipy * Update min scipy * Update min scipy * Update min scipy * Update skimage * Review comments#1 * Update skimage * Revert rename to nwpu * Revert renames * Update min scipy * Fix data dirname * More fixes * Bump numpy minor version * Update scipy * Going back to what worked * Ignore numpy warning * Edit * Edit warning * Ignore another warning * Insanity * Fix pooch * Try older skimage * Try older scipy * scipy 1.2.0 * scipy 1.4.0 * scipy 1.5.0 * scipy 1.4.2 * scipy 1.6.0 * scipy 1.5.4 * scipy 1.5.2 * scipy 1.5.1 * scipy 1.5.1 is new min dep version * Fix skip * Newer scipy * scipy 1.6.3 * scipy 1.6.0 * scipy 1.6.1 * scipy 1.6.2 * scipy 1.6.2 is new min Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
Part 1 of splitting #798 into separate PRs.
Summary of changes:
nwpu.py
tovhr10.py
.scikit-image
dependencyscipy
Example plots:
Ground truth and predicition (object detection training)
Controlling features to be shown and their respective opacity
Closes #894