Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Early stopping based on global acceptance rate #147

Closed
wants to merge 0 commits into from

Conversation

ThibeauWouters
Copy link

As mentioned in #121 and also apparent from my own experiments, the global acceptance rate is actually the best indication of the quality/convergence of a run. Therefore, I have added a few lines of code that implements early stopping based on reaching a desired target global acceptance rate. I have done this in a bit of a hacky way, so feel free to provide feedback or thoughts on how to best approach something like this.

@ThibeauWouters
Copy link
Author

Hi @kazewong , did you get a chance to have a look at this PR? Please do let me know if you want me to make further changes.

@kazewong
Copy link
Owner

@ThibeauWouters Somehow this flew under my radar. I will have a look in the coming week

@kazewong
Copy link
Owner

I like the idea of early stopping.

Tho I am going to do a big round of clean up for flowMC, and I think it would be easier for me to add it in after afterward. Instead of merging this PR, I will mark you as a contributor for the early stopping part in the refactoring.

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