Closed ThibeauWouters closed 6 months ago
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.
@ThibeauWouters Somehow this flew under my radar. I will have a look in the coming week
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.
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.