-
Notifications
You must be signed in to change notification settings - Fork 289
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
Centered option #617
Centered option #617
Conversation
Codecov Report
@@ Coverage Diff @@
## master #617 +/- ##
==========================================
+ Coverage 93.85% 93.93% +0.08%
==========================================
Files 31 31
Lines 6457 6526 +69
Branches 692 712 +20
==========================================
+ Hits 6060 6130 +70
Misses 258 258
+ Partials 139 138 -1
Continue to review full report at Codecov.
|
Fantastic work! |
@marcus7070 I re-ran one of the Azure CI checks. It had failed because it said it couldn't reach github.com. All the checks pass now. Let us know when it's ready for review. Thanks! |
Change all docstrings for centered options to be consistent and more clear
Ready for review now @jmwright. |
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.
Looks good. Thanks @marcus7070
I've requested a review from @adam-urbanczyk as well.
Thanks, looks good @marcus7070 ! |
* Added 2d centered option to rect * Added single bool center option to box * Add single bool center option to sphere * Added single bool center option to wedge * Added 2d centered option to rarray * Black fix * Centered docstrings Change all docstrings for centered options to be consistent and more clear * Docstring fixes * Fix type hint on wedge method
Will resolve #567.
Also fixes some errors in the HTML docs for box and wedge.