Skip to content
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

[Occupancy] Add LoadOccGTFromFile func; Fix wrong occ pkl path #215

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

qihao-plus
Copy link
Contributor

@qihao-plus qihao-plus commented Apr 28, 2023

This PR adds the lost LoadOccGTFromFile function and also tries to fix wrong occ pkl path in occupancy config file.
With this PR, the bevdet-occ-stbase-4d-stereo-512x1408-24e.py can be trained successfully.

2023-04-28 14:48:16,890 - mmdet - INFO - Epoch [1][50/1759] lr: 4.915e-05, eta: 10 days, 12:42:55, time: 5.174, data_time: 0.198, memory: 15119, loss_depth: 0.2679, loss_occ: 2.6189, loss: 2.8868, grad_norm: 4.7577

image

@qihao-plus
Copy link
Contributor Author

qihao-plus commented Apr 28, 2023

Also, the bevdet-occ-r50-4d-stereo-24e.py requires bevdet-r50-4d-stereo-cbgs.pth which is not provided in baidu link @HuangJunJie2017

@qihao-plus
Copy link
Contributor Author

The defaults extra_tag is bevdetv2-nuscenes in https://github.com/HuangJunJie2017/BEVDet/blob/dev2.1/tools/create_data_bevdet.py#L140

@HuangJunJie2017
Copy link
Owner

Also, the bevdet-occ-r50-4d-stereo-24e.py requires bevdet-r50-4d-stereo-cbgs.pth which is not provided in baidu link @HuangJunJie2017

fixed

@HuangJunJie2017
Copy link
Owner

The defaults extra_tag is bevdetv2-nuscenes in https://github.com/HuangJunJie2017/BEVDet/blob/dev2.1/tools/create_data_bevdet.py#L140

yes, should use bevdetv2-nuscenes instead

@HuangJunJie2017 HuangJunJie2017 merged commit e61f196 into HuangJunJie2017:dev2.1 Apr 28, 2023
@HuangJunJie2017
Copy link
Owner

HuangJunJie2017 commented Apr 28, 2023

please pull the latest code to fixe the training occupancy with bev flip augmentation @qihao-plus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants