-
Notifications
You must be signed in to change notification settings - Fork 120
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 some more functionality, docs and tests for root systems #3191
Conversation
f45a0fc
to
00ff8d5
Compare
Please add a property test to make sure that the roots of the dual root system correspond to the coroots of the base root system. |
Returns the `i`-th coroot of `R`, i.e. the `i`-th root of the dual root system of `R`. | ||
This is a more efficient version for `coroots(R)[i]`. | ||
|
||
Also see: `coroots`. |
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.
How about using an actual ref?
Also see: `coroots`. | |
Also see: [`coroots`](@ref). |
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.
This does only work once we have a markdown page collecting the docstrings. That's something @felix-roehrich wants to approach in the near future.
done |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #3191 +/- ##
========================================
Coverage 81.27% 81.27%
========================================
Files 556 557 +1
Lines 74019 74133 +114
========================================
+ Hits 60158 60253 +95
- Misses 13861 13880 +19
|
Co-authored-by: Felix Röhrich <47457568+felix-roehrich@users.noreply.github.com>
can we get a merge of this? maybe @thofma ? |
No description provided.