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

Use MultiDomainFunction in IntegratePEaks1DProfile to share peak parameters across pixels #38439

Open
RichardWaiteSTFC opened this issue Nov 25, 2024 · 0 comments · May be fixed by #38515
Open
Assignees
Labels
Diffraction Issues and pull requests related to diffraction ISIS Team: Diffraction Issue and pull requests managed by the Diffraction subteam at ISIS
Milestone

Comments

@RichardWaiteSTFC
Copy link
Contributor

Is your feature request related to a problem? Please describe.
IntegratePeaks1DProfile seems to underestimate the error on the peak intensity
image

This is possibly due to weaker pixels not being included as the fit is less reliable - as seen for this peak
image

These pixels contribute more to the error than the overall intensity.

Describe the solution you'd like
Use MultiDomainFunction in IntegratePEaks1DProfile to share peak parameters across pixels to make fit of weak pixels in particular more robust.

@RichardWaiteSTFC RichardWaiteSTFC added this to the Release 6.12 milestone Nov 25, 2024
@RichardWaiteSTFC RichardWaiteSTFC self-assigned this Nov 25, 2024
@RichardWaiteSTFC RichardWaiteSTFC added Diffraction Issues and pull requests related to diffraction ISIS Team: Diffraction Issue and pull requests managed by the Diffraction subteam at ISIS labels Nov 25, 2024
@RichardWaiteSTFC RichardWaiteSTFC moved this to In Progress in ISIS Diffraction Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Diffraction Issues and pull requests related to diffraction ISIS Team: Diffraction Issue and pull requests managed by the Diffraction subteam at ISIS
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

1 participant