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

How do we use the variant of your current models : Ours -im-l2z and Ours-im-pc2z #1

Open
prashkmr opened this issue Aug 13, 2020 · 10 comments

Comments

@prashkmr
Copy link

Hi @ChrisWu1997

Can you please tell me how does one use the 2 variants of your models: Ours -im-l2z and Ours-im-pc2z.
Is there any flag or different models made for it, or does one need to manually change the model for running these variants,

I tried searching for some ready-made configuration that can help run these but could not find anyone.

Thanks,
Prashant

@ChrisWu1997
Copy link
Owner

Thanks for taking interest in our paper. Currently the code doesn't support the 2 variants of our model. If you are not in a hurry, I'll update the code to support the 2 variants within a week and inform you once it's done.

@prashkmr
Copy link
Author

prashkmr commented Aug 14, 2020 via email

@ChrisWu1997
Copy link
Owner

I see. In this case, I can send you the code for our 2 variants independently (not integrated into this repo). On which dataset do you need? Or it doesn't matter if you are using your own dataset?

@prashkmr
Copy link
Author

prashkmr commented Aug 14, 2020 via email

@ChrisWu1997
Copy link
Owner

ChrisWu1997 commented Aug 14, 2020 via email

@prashkmr
Copy link
Author

prashkmr commented Aug 14, 2020

Thanks, Chris for this early resolution.
However, I do not see any code folders attached. Can you confirm whether it is in the mail or in the github repo.

@ChrisWu1997
Copy link
Owner

I just resend the code directly to your gmail, please check:)

@prashkmr
Copy link
Author

Got it, thanks to you, Chris 👍

@prashkmr
Copy link
Author

prashkmr commented Aug 14, 2020

Hi Chris,

In the files sent , there is a file common.py that mentions.
self.ae_modelpath = "/mnt/disk4/wurundi/multimodal-pc-completion/3depn-table-pointnetAE-small-2K/model/ckpt_epoch3000.pth"

Does this models refer to the ae or the have trained using the complete Point Clouds?
From your paper, it appears that you use the VAE trained using completely PC in the 2 variants.

Needed a clarification as to whether the line 👍
self.ae_modelpath = "/mnt/disk4/wurundi/multimodal-pc-completion/3depn-table-pointnetAE-small-2K/model/ckpt_epoch3000.pth"

refers to the AE or the VAE trained previously.

Moreover, the 2 variants do not have any scripts for GAN, from which I guess the values of the command line arguments are the default ones assigned in common.py. Correct if I am wrong.

-Prashant

-Prashant

@ChrisWu1997
Copy link
Owner

ChrisWu1997 commented Aug 15, 2020 via email

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