-
Notifications
You must be signed in to change notification settings - Fork 196
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
multihead fine-tuning starting from a previously fit model gives a tensor size error (E0 related?) #622
Comments
@ilyes319 have you had any chance to look at this? |
@bernstei I should have fixed that in the main branch. Could you try and tell me if it is fixed indeed. |
The real code that tries to do this still fails. Let me see what's going on with my reproducible example above. |
When I try to run the code from the original description and the current
Note that I'm not sure that's the error that the real code is getting, but the example above should be working regardless. |
Is that the full error? Just to check, can you confirm you are not using the "compiled" model but the vanilla model for restarting the finetuning. |
Dunno - it works for me. Are you using some weird ASE version? Can you post the full error? Those are all a normal extxyz files.
|
By the way, in case it's helpful, the real code (not the little toy example above) gives a different error. My data only has Z = 28 (Ni), so the Z=12 issue must have to do with the PT head.
|
Ok thank you, I think these are two seperate bugs. Re on ASE, the only weird thing of my version is that I am on windows, that might the problem. I ll try on a linux machine. |
I guess it could be a EOL mismatch type issue. |
Running
the second fit gives the error
xyz files are here, with and extra
.txt
suffix to allow for github upload[edited - replaced txt files with a single zip]
_MACE_files.zip
The text was updated successfully, but these errors were encountered: