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

ITG2e: after author proof corrections #80

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nautolycus
Copy link

Suggested update to dictionary after receiving author proof corrections for International Tables G 2nd edition.

(1) Examples in category TOPOL imported from new topo_examples.cif file.

(2) Numerous small style changes (spelling, punctuation, addition of titles to references). These should be uncontroversial.

(3) Removal of _topol_link.net_id and _topol_net.overall_topology - these may need double-checking with authors.

Brian McMahon and others added 2 commits February 20, 2024 12:16
to Chapters 3.9 and 4.9 of International Tables G 2nd edition; some
minor style changes to text of definitions; examples imported from
external file topo_examples.cif.
    to Chapters 3.9 and 4.9 of International Tables G 2nd edition; some
    minor style changes to text of definitions; examples imported from
    external file topo_examples.cif.
Copy link
Contributor

@vaitkus vaitkus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR. Since it is your first PR on this repository, the automated checks did not trigger and have to be approved by a person with a maintainer role (@jamesrhester should be able to help with this).

I do not have such a role on this repo, however, I was able to run the same checks locally on my machine and preemptively suggest a few changes to the PR. It is probably most convenient to accept or refuse these using the GitHub web interface.

Changes that I proposed in the Topology.dic file mainly deal with the deviations from the IUCr dictionary style guide (see https://github.com/COMCIFS/comcifs.github.io/blob/main/accepted/ddlm_dictionary_style_guide.md).

The data_ header proposed for the topo_examples.cif file is needed to make it a valid CIF file. Without the header some of the automated checks fail to run properly.

Topology.dic Outdated Show resolved Hide resolved
Topology.dic Outdated Show resolved Hide resolved
Topology.dic Outdated Show resolved Hide resolved
topo_examples.cif Show resolved Hide resolved
Topology.dic Show resolved Hide resolved
Thanks for these style fixes - Brian

Co-authored-by: Antanas Vaitkus <[email protected]>
Topology.dic Outdated Show resolved Hide resolved
Agreed that this distinguishes this version cleanly from the version published on the IUCr website.

Co-authored-by: Antanas Vaitkus <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants