-
Notifications
You must be signed in to change notification settings - Fork 33
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
Wrap box with molbox's API #565
base: main
Are you sure you want to change the base?
Conversation
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
lengths = _validate_lengths(lengths) | ||
angles = _validate_angles(angles) | ||
print(lengths, angles) | ||
precision = int(precision or 6) |
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.
Does this set precision to 6 if you cant typecast precision to an int?
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.
I think it does because you have handled that in Molbox.
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.
Ah true, that makes sense.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #565 +/- ##
==========================================
- Coverage 92.44% 90.53% -1.91%
==========================================
Files 66 56 -10
Lines 7024 4439 -2585
==========================================
- Hits 6493 4019 -2474
+ Misses 531 420 -111 ☔ View full report in Codecov by Sentry. |
Closes #563.