Skip to content

Implemented early stopping by specified score and threshold#32

Merged
hdieckhaus merged 2 commits into
Kuhlman-Lab:mainfrom
gezmi:main
Mar 26, 2026
Merged

Implemented early stopping by specified score and threshold#32
hdieckhaus merged 2 commits into
Kuhlman-Lab:mainfrom
gezmi:main

Conversation

@gezmi
Copy link
Copy Markdown
Contributor

@gezmi gezmi commented Jan 26, 2026

added flags to implement early stopping, as an example:
--early_stop_metric iptm --early_stop_threshold 0.85

Setting the metric will also change this metric to be the one used for ranking_score

In case someone wants to change the ranking score, without early stopping, one can use for example:
--early_stop_metric ptm --early_stop_threshold 1.0 => this will result in ptm being used for ranking amongst the models

@hdieckhaus
Copy link
Copy Markdown
Contributor

@gezmi thanks for the PR! Looks like the CI tests are good, I just want to understand what this PR is doing before merge.

When running with multiple seeds, this stops after any seed that hits a certain score threshold? So then it won't run any further seeds for that sequence? I am curious the reasoning for doing this, rather than doing the early stopping after a certain number of recycles.

@gezmi
Copy link
Copy Markdown
Contributor Author

gezmi commented Feb 3, 2026 via email

@hdieckhaus hdieckhaus self-assigned this Mar 26, 2026
Copy link
Copy Markdown
Contributor

@hdieckhaus hdieckhaus left a comment

Choose a reason for hiding this comment

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

Sorry for the delay on this, merging now.

@hdieckhaus hdieckhaus merged commit 6075812 into Kuhlman-Lab:main Mar 26, 2026
1 check passed
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