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

[Bug] Inconsistency with neuralpredictors with loading SOTA sensorium model weights #117

Open
KonstantinWilleke opened this issue Apr 11, 2024 · 1 comment
Assignees

Comments

@KonstantinWilleke
Copy link
Collaborator

@MaxFBurg After all the issues were fixed with independent_bn_bias, we have a new problem: The code works as intended, but it is not backwards compatible. If Sensorium and neuralpredictors with the most recent commits is installed, the model weights of the SOTA models can't be loaded.
It's not clear to me yet what the best way forward is, so I'm just doing the issue for now and maybe we can think of a solution next week.

@MaxFBurg
Copy link
Member

MaxFBurg commented Apr 12, 2024

@KonstantinWilleke is it about initializing the model (i.e. the notebooks do not initialize the same model architecture anymore - I think we tested this though when making these changes) or does loading the weights fail (i.e. the named parameters of the model do not match the names/keys in the saved state_dict)?

If you need a fast (and easy) solution installing a previous commit should work. There are certainly also other solutions, let's think what would be best for what you need.

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