Skip to content

Apply all intensity corrections after SNIP in polar #1776

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

Merged
merged 3 commits into from
Apr 16, 2025

Conversation

psavery
Copy link
Collaborator

@psavery psavery commented Jan 14, 2025

No description provided.

@psavery psavery force-pushed the polar-absorption-correction branch from 6eeec95 to 8b86c0b Compare March 6, 2025 17:13
@psavery psavery changed the title Apply absorption correction after SNIP in polar Apply all intensity corrections after SNIP in polar Mar 6, 2025
@psavery psavery force-pushed the polar-absorption-correction branch 5 times, most recently from 2477506 to 4ea30e9 Compare March 7, 2025 18:33
@psavery
Copy link
Collaborator Author

psavery commented Mar 19, 2025

Note for self: I need to make similar changes for the median filter when #1761 is merged.

psavery added 3 commits April 15, 2025 16:37
Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
We will just now combine all intensity corrections together, warp
them to the polar view, and apply them after SNIP...

Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
Otherwise, we cannot apply the intensity corrections...

Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
@psavery psavery force-pushed the polar-absorption-correction branch from 1e8a1b4 to e7b29f4 Compare April 15, 2025 21:52
@saransh13
Copy link
Member

There are fixes in Hexrd branch HEXRD/hexrd#793 that are needed to be merged before this branch works. Otherwise LGTM.

@psavery psavery merged commit 2092bec into master Apr 16, 2025
9 checks passed
@psavery psavery deleted the polar-absorption-correction branch April 16, 2025 03:13
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