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

Latest ASE giving problems #17

Open
danieleongari opened this issue Feb 3, 2021 · 3 comments
Open

Latest ASE giving problems #17

danieleongari opened this issue Feb 3, 2021 · 3 comments

Comments

@danieleongari
Copy link
Collaborator

With the latest version of ASE (3.21.1) I get the following error when loading a standard CIF:

  File "/Users/danieleongari/opt/anaconda3/lib/python3.8/site-packages/ase/io/cif.py", line 499, in parse_block
    assert line.lower().startswith('data_')
AssertionError

the problem does not occur with my previous version of ASE (3.19.1), so maybe let's give a look or constrain the version.

@ltalirz
Copy link
Member

ltalirz commented Feb 3, 2021

we've constrained to ase<3.20 in aiida-lsmo already

at some point we'll need to figure out "who's right" (or whether there is a way to make the new ase more lenient)

@ltalirz
Copy link
Member

ltalirz commented Feb 3, 2021

from the release notes:

3.20.2:

  • CIF writer now uses up-to-date variable definitions from the CIF standard instead of deprecated ones. Also, it no longer writes columns of dummy data that doesn’t depend on the atoms.

Perhaps the reason is that we are using variable definitions that are deprecated in the CIF standard.

@ltalirz
Copy link
Member

ltalirz commented Feb 3, 2021

Change in the source code: https://gitlab.com/ase/ase/-/merge_requests/2253/diffs
particularly affects space group symmetry symbols

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