-
Notifications
You must be signed in to change notification settings - Fork 35
[WIP] Net charge support in ABFEs #1614
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1614 +/- ##
==========================================
- Coverage 95.32% 92.80% -2.52%
==========================================
Files 183 183
Lines 15765 15803 +38
==========================================
- Hits 15028 14666 -362
- Misses 737 1137 +400
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
No API break detected ✅ |
|
I am very interested in this feature, and I appreciate the hard work that is being done on this package in general. Is the code here ready for testing / do you have a timeline for when it might be released? |
|
It's been a while since I had a chance to work on this, looking at the tests it seems like there were a few things that needed fixing - probably not an overwhelming amount for a first draft. I'll bring this up for discussion internally - one of the biggest blockers we had with this was that we just didn't have enough compute to run tests / benchmark. Is this something you might be interested in helping out with? |
|
Hi @IAlibay, We could be interested in running tests / benchmarking, but we would want to know what these tests/benchmarks would look like and how computationally expensive you expect them to be. |
Fixes: #1662
Checklist
newsentryDevelopers certificate of origin