-
Notifications
You must be signed in to change notification settings - Fork 3
HTF speed up #128
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
HTF speed up #128
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #128 +/- ##
=======================================
Coverage 86.86% 86.87%
=======================================
Files 11 11
Lines 1439 1440 +1
=======================================
+ Hits 1250 1251 +1
Misses 189 189 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #128 +/- ##
=======================================
Coverage 86.86% 86.87%
=======================================
Files 11 11
Lines 1439 1440 +1
=======================================
+ Hits 1250 1251 +1
Misses 189 189 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
IAlibay
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
GPU runners aren't working, but I've tested most of the GPU tests locally and they seem to be passing. |
ijpulidos
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this. LGTM!
(cherry picked from commit 73f74e1)
Porting some of the fixes from OpenFreeEnergy/openfe#1566 to make sure HTF tests still pass till we can migrate the tests.