-
Notifications
You must be signed in to change notification settings - Fork 16
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
✨ Add more labels for Geometry Parameterisations #3081
Conversation
5a37d40
to
a3511be
Compare
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3081 +/- ##
===========================================
- Coverage 79.77% 79.71% -0.06%
===========================================
Files 227 227
Lines 25478 25512 +34
===========================================
+ Hits 20324 20337 +13
- Misses 5154 5175 +21 ☔ View full report in Codecov by Sentry. |
ffac2af
to
c4e7605
Compare
c4e7605
to
2db5691
Compare
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.
Yeah LGTM!
This is a proper nit, but I don't like the double arrows used (personally)
On line 319, I would change arrowstyle to
"arrowstyle": "<|-",
Quality Gate passedIssues Measures |
* ✨ Add more labels for sextuple arc * one directional arrow
Linked Issues
Closes #1747 (well as many as is reasonable)
Description
Add some of the missing labels to our geometry parameterisations
Interface Changes
Checklist
I confirm that I have completed the following checks:
pytest tests --reactor
pre-commit run --from-ref develop --to-ref HEAD
sphinx-build -W documentation/source documentation/build