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

Made compatible with torchsparse1.4 #63

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AbdelazizBouzidi
Copy link

@AbdelazizBouzidi AbdelazizBouzidi commented Mar 24, 2023

Hello,

I'm a computer vision student, i'm actually doing a final year research project on the subject of 3D reconstruction based on a single image, I find your work astonishing, i congratulate your for that.

In this pull request, i have fixed some compatibility issues with the latest version of torchsparse (1.4 i guess), and also the latest version of pytorch and cuda11.7.

I have seen in one of the issues answers that the training code for the PCMs aren't released because of IP issues, it is completely understandable but i wonder if you might share with me how did you procede, with regard to the data generation process, the magnitude of the perturbations applied to the focals and the depth, what cost function have you used and any idea or intuition about the hyper parameters.

Your answer would be very much appreciated.

@AbdelazizBouzidi AbdelazizBouzidi changed the title Made compatible with torchsparse2.0 Made compatible with torchsparse1.4 Mar 24, 2023
@CaiSir0075
Copy link

Can you tell me if the version you are using can run 3D reconstruction properly? The torchsparse I downloaded seems to be missing some files (crying)

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