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

Should the _atom_sites_moment_Fourier.axes_description be deprecated in favour of ATOM_SITES_AXES? #80

Open
vaitkus opened this issue Mar 18, 2024 · 2 comments

Comments

@vaitkus
Copy link
Collaborator

vaitkus commented Mar 18, 2024

The description of the _atom_sites_moment_Fourier.axes_description data item notes, that this data item is analogous to the _atom_sites_displace_Fourier.axes_description and _atom_sites_rot_Fourier.axes_description data items from the msCIF dictionary [1]. However, both of these names are now marked as deprecated in that dictionary in favour of the ATOM_SITES_AXES category. If I understand correctly, it seems that this category allows to list one or more transformation matrices (the _atom_sites_axes.matrix data item) optionally paired with the human-readable description of the transformation (the _atom_sites_axes.transf_description data item). Maybe the _atom_sites_moment_Fourier.axes_description item should also be deprecated in a similar way?

[1] https://github.com/COMCIFS/Modulated_Structures

@brantonc
Copy link
Collaborator

brantonc commented Mar 18, 2024 via email

@jamesrhester
Copy link
Contributor

Given Branton's comment I agree that we should harmonise with the msCIF approach.

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

3 participants