Skip to content

Update edges#9

Merged
mretegan merged 8 commits intomainfrom
update-edges
Feb 26, 2026
Merged

Update edges#9
mretegan merged 8 commits intomainfrom
update-edges

Conversation

@mretegan
Copy link

Added a few missing ones and a short definition.

@mretegan mretegan self-assigned this Feb 23, 2026
@maurov
Copy link

maurov commented Feb 26, 2026

@mretegan thanks for your work on this.

For the edges, I think we should allow only the IUPAC notation. Contrary to the emission lines, it is rather simple to identify the edge in IUPAC notation. The ASCII notation is misleading because it can be either written in LaTeX or another way. I would keep the ASCII notation in the documentation, but the enumeration should keep only IUPAC.

Again, I find clear how is stated in the XrayDB documentation: https://xraypy.github.io/XrayDB/overview.html#x-ray-edges

@mretegan
Copy link
Author

@maurov Okay, I updated the definition.

Copy link

@maurov maurov left a comment

Choose a reason for hiding this comment

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

OK to merge!

@mretegan mretegan merged commit f688001 into main Feb 26, 2026
3 checks passed
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