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.

Sign up for GitHub

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

Jump to bottom

Change name of Arrays that does not conform to AiiDA naming convention #42

Closed
JPchico opened this issue Jun 10, 2022 · 1 comment
Closed

Change name of Arrays that does not conform to AiiDA naming convention #42

JPchico opened this issue Jun 10, 2022 · 1 comment
Labels
bug
Milestone
Refactoring and g...

Comments

@JPchico
Copy link
Collaborator

JPchico commented Jun 10, 2022

The names generated by lammps, might result in array names that are not suitable for aiida. Hence one needs to change the names of the arrays so that they can be stored in the database without problem.

This tends to happen when one is trying to store arrays that have [ in the name.
@JPchico JPchico added the bug label Jun 10, 2022
JPchico pushed a commit to JPchico/aiida-lammps that referenced this issue Jun 10, 2022
Fixing an issue in which the arrays names might not be compatible wit…
b00ce20 
…h what AiiDA enforces. For that the `[` and `]` that are usually given by lammps to arrays components are replaces by `__`. Fixes aiidaplugins#42.
@JPchico JPchico mentioned this issue Jun 10, 2022
Merged
@JPchico
Copy link
Collaborator Author

JPchico commented Jun 16, 2022

Closed via #43

@JPchico JPchico closed this as completed Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Refactoring and generalization
Development

No branches or pull requests
1 participant
@JPchico