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

Question about the this baseline #38

Open
zkyyyyyyy opened this issue Apr 29, 2024 · 6 comments
Open

Question about the this baseline #38

zkyyyyyyy opened this issue Apr 29, 2024 · 6 comments

Comments

@zkyyyyyyy
Copy link

Hi, thanks to your great work.
I found the baseline code of this Challenge based on bevformer you provide does not have alignment operation between prev bev and current bev. In bevformer repository it is defined in "transformer.py". But in "transformerOcc.py" I do not find it. So can I ask why?

8ed535ae1a0db732e5235fac86233a2f
@sephyli
Copy link
Contributor

sephyli commented Apr 30, 2024

Hello! The code in this repository builds upon last year's baseline, which deliberately omitted the temporal component of BEVFormer.
It appears that the unresolved issue stems from BEVFormer's shift calculations being based on a y-forward coordinate system, while for the occupancy dataset, we utilize the nuScenes ego coordinate system, which is x-forward. Previous contributors likely did not have the time to make the necessary adjustments and opted to remove this feature for convenience.

@sephyli
Copy link
Contributor

sephyli commented Apr 30, 2024

We have a fix about the coordinates issue at LaneSegNet repo. You can check it out here.

@zkyyyyyyy
Copy link
Author

We have a fix about the coordinates issue at LaneSegNet repo. You can check it out here.

I try to run this code but got no key named "can_bus" in the dict ['img_metas']. Then I go to check the anno .pkl file, actually no 'can_bus' information in it. Can you help me with this issue?

@sephyli
Copy link
Contributor

sephyli commented May 1, 2024

The OpenOcc dataset doesn't construct a can_bus tensor. You can canstruct one or modify the code as you like.

@sephyli
Copy link
Contributor

sephyli commented May 1, 2024

Here is a doc for can bus.

@Icecream-blue-sky
Copy link

The OpenOcc dataset doesn't construct a can_bus tensor. You can canstruct one or modify the code as you like.

So will the current baseline(pre_bev is None by default) perform worse than the case using and aligning pre_bev ?

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

No branches or pull requests

3 participants