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

Update nomenclature for SESANS plots #2663

Open
gnsmith opened this issue Oct 17, 2023 · 1 comment · Fixed by #2754
Open

Update nomenclature for SESANS plots #2663

gnsmith opened this issue Oct 17, 2023 · 1 comment · Fixed by #2754
Assignees
Labels
Enhancement Feature requests and/or general improvements

Comments

@gnsmith
Copy link

gnsmith commented Oct 17, 2023

I am the chair of the canSAS SESANS Reproducibility Working Group.

After much discussion in our group, we recently agreed recommendations for nomenclature for SESANS measurements. This is an update of previous nomenclature, but we feel that these should stick for now. Hopefully! There was at least agreement from the three facilities that run SESANS instruments...

The paragraph about it is on our wiki page.
https://wiki.cansas.org/index.php?title=SESANS_Reproducibility_Working_Group

There are two main changes that would need to be made in SasView. The variable for spin-echo length needs to updated from z to δ. The unit for both length scale and wavelength needs to be changed from A to Å.

I just checked and the Å (ångström) rather than A (amp) thing is also the case in the normal SAS window.

@gnsmith gnsmith added the Enhancement Feature requests and/or general improvements label Oct 17, 2023
@caitwolf caitwolf self-assigned this Jan 18, 2024
@caitwolf
Copy link
Contributor

Changing units from A to Å would be a larger change that affects more than SESANS data so I've created another issue #2753.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Feature requests and/or general improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants