Skip to content

BUG: Use max corner angle in Parker weighting warning check - #999

Open
axel-grc wants to merge 1 commit into
RTKConsortium:mainfrom
axel-grc:ShortScan
Open

axel-grc wants to merge 1 commit into
RTKConsortium:mainfrom
axel-grc:ShortScan

Conversation

@axel-grc

Copy link
Copy Markdown
Collaborator

With source/detector offsets the fan is asymmetric, so delta must exceed the maximum corner half-beam angle, not the minimum. Also report that angle in the warning message.

Fix #393

With source/detector offsets the fan is asymmetric, so delta must exceed the maximum corner half-beam angle, not the minimum. Also report that angle in the warning message.

Fix RTKConsortium#393
@SimonRit
SimonRit self-requested a review September 14, 2026 06:04

@SimonRit SimonRit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, but for me the original code made more sense. Can you reproduce #393? I can't so I wonder if this issue is still relevant

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.

Parker weighting issues no warning with large source offsets

2 participants