-
Notifications
You must be signed in to change notification settings - Fork 53
Correct rmsds #604
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
Correct rmsds #604
Conversation
Codecov ReportBase: 73.35% // Head: 73.31% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #604 +/- ##
==========================================
- Coverage 73.35% 73.31% -0.04%
==========================================
Files 107 107
Lines 6989 6979 -10
==========================================
- Hits 5127 5117 -10
Misses 1862 1862
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
Great catch!
| Q_l = Q[l_start: l_end + 1] | ||
| P_l = P[l_start: l_end + 1] |
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.
oops! hehe
You are about to submit a new Pull Request. Before continuing make sure you read the contributing guidelines and you comply with the following criteria:
toxtests pass. Runtoxcommand inside the repository folder-test.cfgexamples execute without errors. Insideexamples/runpython run_tests.py -bCloses #603 by
haddock3/src/haddock/modules/analysis/caprieval/capri.py
Lines 181 to 182 in f0ebd69
haddock3/src/haddock/modules/analysis/caprieval/capri.py
Lines 202 to 203 in f0ebd69
haddock3/src/haddock/modules/analysis/caprieval/capri.py
Lines 177 to 178 in f0ebd69