-
Notifications
You must be signed in to change notification settings - Fork 46
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
Training dataset is wrongly thought to only have two datapoints #79
Comments
I would suggest you to try iterating through the structures in your For instance,
Meanwhile, I notice your |
I did that test and it says that both my .xyz that I read in to create the .extxyz, and the .extxyz, have four structures. Here is the .xyz I used:
And here is the output of the code you gave me above to count and check frames:
Here is my code for reading the .xyz file and making the .extxyz:
And here is the .extxyz file made by the above code. I still get the error where Nequip thinks there are only two datapoints.
And here is the output of the frame counting code you gave me above:
|
Thanks for checking this! I just ran a training with your 4-structure |
No problem. It works when I do that, thanks for all your help! Just in case anyone else has this problem in the future, I did the following: Fix the .extxyz so it has energy (originally I still had this problem when the energy tag was there) Make a new environment. conda install pytorch==1.10 git clone https://github.com/mir-group/nequip.git git clone https://github.com/mir-group/allegro.git My environment looks like this:
|
This is related to this discussion:
#74
As advised there, I used ase.io.read and ase.io.write to make my .extxyz training set our of ase atoms, but when I run nequip-train I get the following error:
I modified nequip/train/trainer.py so that it would print out total_n which is the number of datapoints. This comes out as 2. However, my dataset has 249 datapoints in it. Here are the first 2 as an example:
Here is the relevant part of my config yaml:
What's wrong with my input? I have 249 structures but it thinks there are 2. I've checked all the docs for ase, Nequip and Allegro and it looks OK. This case has no PBC but when I run it with lattice parameters (physically valid ones) and pbc="T T T" it still gives the same error. I'm sure I've made an error somewhere but I can't find it.
Thanks in advance.
The text was updated successfully, but these errors were encountered: