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

Powder calibrate with widget #2016

Merged
merged 61 commits into from
Mar 10, 2022
Merged

Powder calibrate with widget #2016

merged 61 commits into from
Mar 10, 2022

Conversation

elena-pascal
Copy link
Contributor

Use pyFAI calibrate for electron powder diffraction.
Since pyFAI was developed for X-rays the expected ring density is lower than that of electron powder patterns. To circumvent this I proposed using a matplotlib widget to guesstimate an initial beam geometry.

Would be worth in the future to look at pyFAI's spot picker instead to feed the algorithm the correct ring instead.

@elena-pascal elena-pascal requested a review from dagewa March 7, 2022 16:45
@codecov
Copy link

codecov bot commented Mar 9, 2022

Codecov Report

Merging #2016 (ffa594d) into main (1f6bc5c) will decrease coverage by 0.26%.
The diff coverage is 1.34%.

@@            Coverage Diff             @@
##             main    #2016      +/-   ##
==========================================
- Coverage   68.55%   68.29%   -0.27%     
==========================================
  Files         651      653       +2     
  Lines       75519    75816     +297     
  Branches    10790    10820      +30     
==========================================
+ Hits        51771    51777       +6     
- Misses      21734    22026     +292     
+ Partials     2014     2013       -1     

@ndevenish ndevenish enabled auto-merge (squash) March 10, 2022 12:38
@ndevenish ndevenish merged commit a25285e into main Mar 10, 2022
@ndevenish ndevenish deleted the powder_calibrate branch March 10, 2022 13:35
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.

None yet

3 participants