Skip to content

Conversation

@Tom-Willemsen
Copy link
Member

@Tom-Willemsen Tom-Willemsen commented Oct 2, 2025

Description of work

  • Implement flood-map correction (to normalise for pixel efficiencies) in reflectometry detector-mapping alignment
  • Improve guess for Gaussian - now uses centre-of-mass as the initial x0 estimate for the fitting routine. This makes the guess better for skewed curves, making the fit more likely to eventually converge reliably.
  • Various minor tweaks as a result of "shake-down" on POLREF

Ticket

No ticket - this PR is the result of "shake-down" testing on POLREF.

Labels

  • 'bluesky-Semver-Minor' - New feature

Acceptance criteria

  • Pull request title is understandable for a user (e.g. scientist) reading the release notes. The PR title should be a short description of the change from a user perspective.
  • Pull request has appropriate labels for automatic release-notes generation
  • POLREF are happy (check with @christykinane, or believe me that I sat on POLREF until they were happy)

Documentation

See PR

@Tom-Willemsen Tom-Willemsen added the bluesky-Semver-Minor New functionality / back-compatible changes label Oct 2, 2025
@Tom-Willemsen Tom-Willemsen moved this to Review in PI_2025_08 Oct 2, 2025
@Tom-Willemsen Tom-Willemsen changed the title Reflectometry: add flood-map correction Reflectometry: add flood-map correction & change Guassian fit to use centre-of-mass as an initial x0 guess Oct 29, 2025
@rerpha rerpha merged commit 90aa070 into main Oct 29, 2025
14 checks passed
@rerpha rerpha deleted the polref branch October 29, 2025 14:23
@github-project-automation github-project-automation bot moved this from Review to Done in PI_2025_08 Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bluesky-Semver-Minor New functionality / back-compatible changes size/L

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants