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

abcd download-ed configs have elements JSON string that messes up gap fitting #105

Open
gelzinyte opened this issue Apr 14, 2021 · 4 comments

Comments

@gelzinyte
Copy link
Contributor

xyz frames from abcd download contain an elements entry (e.g. elements="_JSON {\"6\": 2, \"8\": 1, \"1\": 6}") which messes with reading structures in gap_fit. The simplest solution might be to just not write elements when abcd-downloading? Especially because there's also a formula entry (formula=C2H6O).

@gabor1
Copy link
Contributor

gabor1 commented Apr 14, 2021

hm... the XYZ formatting is in a bit of a flux right now. But some form of complex data type will be there (check out the #extxyz channel on our slack), so maybe we need to teach gap_fit to just treat that as a string and not worry about it.

@gabor1
Copy link
Contributor

gabor1 commented Apr 14, 2021

how does gap_fit fail? there are already random string items in the xyz files that I use and they are fine. maybe it's the quoted quotes?

@gelzinyte
Copy link
Contributor Author

Not sure if it's just the quoted quotes (maybe non-escaped {?), but it looks like the parsing of the info line gets messed up

$ head train.xyz
9
Lattice="50.0 0.0 0.0 0.0 50.0 0.0 0.0 0.0 50.0" Properties=species:S:1:pos:R:3:dft_forces:R:3 normal_mode_temperature=T dft_energy=-4212.509602197413 compound=ethanol n_atoms=9 config_type=ethanol_mol hash=50b506dbeef00fee5a23aceeb7ac0eb9 username=eg475 filename=train.xyz hash_structure=0154021c79357894b4136fd608fefad4 volume=124999.99999999991 mol_or_rad=mol dataset_type=train_compounds_train_configs normal_mode_energy=0.23389851991840674 formula=C2H6O elements="_JSON {\"6\": 2, \"8\": 1, \"1\": 6}" pbc="F F F"
C       -0.93401342       0.07042787       0.05111524       0.35621935      -0.69215077      -0.84583725
C        0.53526377      -0.43666892      -0.11341624      -1.69537623       1.01342896       1.25964530
O        1.47668645       0.40662443       0.51842773       0.09388185      -0.42907879       0.50304526
$ bash sub.sh
Wed 14 Apr 17:27:59 BST 2021
SYSTEM ABORT: Traceback (most recent call last)
File "/opt/womble/QUIP/git_repo/src/libAtoms/xyz.c", line 765 kind IO
Missing value for parameter "6 pbc"

@gabor1
Copy link
Contributor

gabor1 commented Apr 14, 2021 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