alchemistry / alchemlyb

the simple alchemistry library
https://alchemlyb.readthedocs.io
BSD 3-Clause "New" or "Revised" License
189 stars 49 forks source link

ABFE workflow cannot handle 0 or False for forwrev #276

Closed xiki-tempula closed 1 year ago

xiki-tempula commented 1 year ago

if I don't need to do convergence analysis, I will usually do ABFE(forwrev=None) and if I want to do convergence analysis, I will do ABFE(forwrev=10), these two ways of using ABFE are all fine. However, if someone does ABFE(forwrev=0) or ABFE(forwrev=False), the workflow will crash.