-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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] Unify camera poses #653
Conversation
Sorry, will think about failed tests later. |
Codecov Report
@@ Coverage Diff @@
## master #653 +/- ##
==========================================
- Coverage 51.02% 51.01% -0.02%
==========================================
Files 204 204
Lines 15395 15383 -12
Branches 2492 2488 -4
==========================================
- Hits 7856 7848 -8
+ Misses 7021 7017 -4
Partials 518 518
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review 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.
Good job! Overall the code is much cleaner and easy to follow now. Can you kindly post a few visualization of ImVoteNet so that we can make sure the correctness of this PR? (Also some browse_dataset
results on SUN RGB-D multi-modality task if you have time)
Also the CI is broken. Probably not by my commit. |
Yes it's because of MMSeg version. Will raise a PR to fix it. |
Hi @Wuziyi616 , |
Hi @filaPro, please merge master so that the CI failure caused by MMSeg version incompatibility will be solved. Then we can run CI to check your PR's correctness. Thanks. |
@Wuziyi616 Done. |
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.
Great work!
Replace
calib['K']
andcalib['Rt']
withimg_meta['depth2img']
forSUN RGB-D
to be likeimg_meta['lidar2img']
for outdoor datasets. More discussion can be found in #620#issuecomment-859664418.This PR affects visualization on
SUN RGB-D
andImVoteNet
model. I tested these 2 things. ValidationmAP
forSUN RGB-D
improved from64.04
to64.61
.