-
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] Fix bg point filter #609
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.
The issue said this line is also strange. Is it also caused by this bug?
configs/3dssd/3dssd_kitti-3d-car.py
Outdated
@@ -87,8 +87,8 @@ | |||
] | |||
|
|||
data = dict( | |||
samples_per_gpu=4, | |||
workers_per_gpu=4, | |||
samples_per_gpu=2, |
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.
This config file seems strange to me. It doesn't have batch_size x num_GPUs
in its name (such as 2x8
)?
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 forget to modify it, sorry
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.
What about the filename? Should we change it to something like 3dssd_**4x8**_kitti-3d-car.py
?
Codecov Report
@@ Coverage Diff @@
## master #609 +/- ##
==========================================
+ Coverage 50.92% 50.97% +0.04%
==========================================
Files 197 197
Lines 15056 15057 +1
Branches 2444 2445 +1
==========================================
+ Hits 7668 7675 +7
+ Misses 6884 6878 -6
Partials 504 504
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
fix the issue(#347)