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

Dimension mismatch of trained weights (ReSynthdata_12outfit) #16

Open
SuwoongHeo opened this issue Apr 16, 2022 · 3 comments
Open

Dimension mismatch of trained weights (ReSynthdata_12outfit) #16

SuwoongHeo opened this issue Apr 16, 2022 · 3 comments

Comments

@SuwoongHeo
Copy link

Thanks for sharing the code of the wonderful work!

I had tried to test the code with pre-trained weight for ReSynthedata provided in the link in Readme. With the default configuration setting, there is a dimension mismatch between the pre-trained weight and the defined network. Specifically, for unet_posefeat defined by this module, UnetNoCond7DS the layer upconvC5 should have input channel size as 256. However, in the pre-trained model, its dimension is 384 which yields dimension mismatch error during loading that model like below.

image

Is there any way to fix this?

@QUMINGZHEMENAN
Copy link

Has your problem been solve?

@fishfishson
Copy link

it seems the author changed the arch of net in his recent commit f865a01. Restore it and the net works well.
BTW @qianlim is it a bug?

@qianlim
Copy link
Owner

qianlim commented Apr 24, 2022

Hey guys! @fishfishson is right: my latest commit fixed a bug in the original code (= where all the pre-trained checkpoints are based on, also = the model that are generates the quantitative results in the paper). So this fix makes the architecture not compatible with the previous checkpoints. However I've verified that fixing the bug doesn't significantly change the model's performance (the concrete quantitative results are also slightly subject to the random seeds on each machine). So you can either checkout to an earlier commit e.g. a05404d which is compatible with the pre-trained models; or just use the latest commit and re-train the model.

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

4 participants