-
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
mmdet3d coordinate for gt boxes are xyzlwh
, here in CenterFormer, gtboxes[3] should be length not width.
#2360
Comments
@DezeZhao, Hi, thanks for your kind reminder. It's a minor flaw. Fortunately, it has not related to the precision considering the detailed implements among this:
Anyway, could you create a PR to fix it (include here: https://github.com/open-mmlab/mmdetection3d/blob/341ff9980f/mmdet3d/models/dense_heads/centerpoint_head.py#L522) |
yeah, I got it later. It is ok if these 2 values corresponding to x and y axis and keep their order. |
…es(xyz) and dimensions(lwh) correspondence (open-mmlab#2360)
Are you sure this is an issue? length should be larger than width, right? if legth>width: |
In mmdetection3d, gt boxes are organized by |
Prerequisite
Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
master branch https://github.com/open-mmlab/mmdetection3d
Environment
no
Reproduces the problem - code sample
mmdetection3d/projects/CenterFormer/centerformer/centerformer_backbone.py
Line 662 in 341ff99
Reproduces the problem - command or script
mmdetection3d/projects/CenterFormer/centerformer/centerformer_backbone.py
Line 662 in 341ff99
Reproduces the problem - error message
mmdetection3d/projects/CenterFormer/centerformer/centerformer_backbone.py
Line 662 in 341ff99
Additional information
mmdetection3d/projects/CenterFormer/centerformer/centerformer_backbone.py
Line 662 in 341ff99
The text was updated successfully, but these errors were encountered: