Skip to content

Lookup relative error threshold#246

Open
jl-wynen wants to merge 2 commits intomainfrom
lookup-error-threshold
Open

Lookup relative error threshold#246
jl-wynen wants to merge 2 commits intomainfrom
lookup-error-threshold

Conversation

@jl-wynen
Copy link
Member

@jl-wynen jl-wynen commented Mar 9, 2026

No description provided.

@jl-wynen jl-wynen requested a review from nvaytet March 9, 2026 16:51
Copy link
Member

@nvaytet nvaytet left a comment

Choose a reason for hiding this comment

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

I don't understand why CI is failing... a vector coordinate mismatch does not seem to originate from these changes...

email-validator==2.3.0
# via scippneutron
essreduce==26.1.1
essreduce==26.3.1
Copy link
Member

Choose a reason for hiding this comment

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

Should this be set as a lower bound in the pyproject.toml?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@jl-wynen jl-wynen force-pushed the lookup-error-threshold branch from 9a0b187 to edb8ec9 Compare March 10, 2026 06:29
@jl-wynen
Copy link
Member Author

I don't understand why CI is failing... a vector coordinate mismatch does not seem to originate from these changes...

This is related to a change in ESSreduce. Not sure yet which, but the coord transform from time to wavelength no longer removes the position. Do you know why?

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