Skip to content

Move the scale variation check to the operator card creation - #238

Merged
scarlehoff merged 3 commits into
mainfrom
move_the_check_to_the_op
Nov 11, 2025
Merged

scarlehoff merged 3 commits into
mainfrom
move_the_check_to_the_op

Conversation

@scarlehoff

@scarlehoff scarlehoff commented Oct 28, 2025 •

Copy link
Copy Markdown
Member

Instead of waiting for hours for the EKO to be produced only to find out at the end that you couldn't use it.

@felixhekhorn felixhekhorn added the refactor Refactor code label Oct 29, 2025
Comment thread src/pineko/theory.py Outdated
@scarlehoff
scarlehoff force-pushed the move_the_check_to_the_op branch from 4191822 to 4375fad Compare November 4, 2025 10:59
scarlehoff and others added 2 commits November 4, 2025 11:59
Co-authored-by: Felix Hekhorn <felixhekhorn@users.noreply.github.com>
@scarlehoff
scarlehoff force-pushed the move_the_check_to_the_op branch from 4375fad to 0d12492 Compare November 4, 2025 10:59
@scarlehoff

Copy link
Copy Markdown
Member Author

@evagroenendijk I needed to remove the delete to be able to combine different fktables. Not sure whether we just forgot to remove it somehow from your branch or something? (I had rebased this on top of main so it should include your changes)

@scarlehoff

Copy link
Copy Markdown
Member Author

@evagroenendijk can I merge this? (i.e., is the removal of the delete ok?)

@scarlehoff
scarlehoff merged commit f9fcd2c into main Nov 11, 2025
6 checks passed
@scarlehoff
scarlehoff deleted the move_the_check_to_the_op branch August 3, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants