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

Code checking with different version of MinkowskiEngine #22

Closed
tonghe90 opened this issue Feb 4, 2020 · 2 comments
Closed

Code checking with different version of MinkowskiEngine #22

tonghe90 opened this issue Feb 4, 2020 · 2 comments

Comments

@tonghe90
Copy link

tonghe90 commented Feb 4, 2020

I check out the newest version of MinkowskiEngine, but the training process is quite different from the previous version(c52ba2f). Could you please check the code for scannet segmentation

@tonghe90
Copy link
Author

tonghe90 commented Feb 4, 2020

The loss curve is quite different (previous is better), and the newest version fails to load 4 examples in a batch, while the previous is ok

@chrischoy
Copy link
Owner

chrischoy commented Feb 4, 2020

Hi Tong,

Could you pull from the SpatioTemporalSegmentation repository master as well?

The possible cause of the issue is that we prepend the batch indices before the coordinates from ME v0.4 (https://stanfordvl.github.io/MinkowskiEngine/sparse_tensor.html#MinkowskiEngine.SparseTensor.SparseTensor) and I updated the SpatioTemporalSegmentation repository as well.

In general, it is discouraged to directly generate batched coordinates or get decomposed coordinates. Please refer to the warning section of the Sparse Tensor API to make your code up-to-date with the latest Minkowski Engine.

@tonghe90 tonghe90 closed this as completed Feb 4, 2020
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