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

The spconv version #6

Closed
Orbis36 opened this issue Oct 29, 2023 · 4 comments
Closed

The spconv version #6

Orbis36 opened this issue Oct 29, 2023 · 4 comments

Comments

@Orbis36
Copy link

Orbis36 commented Oct 29, 2023

Thanks for your work first.
My question is why your codes cannot be run on the latest SPCONV version(2.3.6).
After several days of debugging, I finally found that the spconv version you use here is weird.

For example,
If you use the .indices method to get the voxel index of the feature map backbone.
In the version you provide, the order of batch dim is increased. In this way, every batch is organized separately.
it looks like (0, x, y, z), (0, x, y, z) , (1, x, y, z), (1, x, y, z) , (2, x, y, z), (2, x, y, z)

However, from 2.2 version, the return value is out of order.
it looks like (0, x, y, z), (1, x, y, z) , (1, x, y, z), (0, x, y, z) , (2, x, y, z), (1, x, y, z)

Could you please check it?
The lastest version of spconv is much faster than 2.1 on Ampere Architecture card

@Orbis36
Copy link
Author

Orbis36 commented Oct 29, 2023

This out of order problem seems will lead to the loss issues, as it will start from 60 when bs=4

@Nightmare-n
Copy link
Owner

Nightmare-n commented Oct 30, 2023

Hi, the used version of spconv_cu111 seems to be 2.1.25 in our environment. Is it possible to sort the indices first to fix this error?

@Orbis36
Copy link
Author

Orbis36 commented Nov 2, 2023

Hi, I have checked the method you mentioned. However, it seems that the loss is still very high even if the order has been sorted. Could you please check it? It's very convenient to change the spconv version and check where is the problem.

@Orbis36
Copy link
Author

Orbis36 commented Nov 2, 2023

Solved, I forgot you also do 1 block of spconv in the head.
And yes, they can be solved by reorder the indices

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

2 participants