-
-
Notifications
You must be signed in to change notification settings - Fork 487
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
Implement basis_of_weight for rings of quasimodular forms #35029
Implement basis_of_weight for rings of quasimodular forms #35029
Conversation
This is a really cool feature. It would be nice if this was used to refactor a bit |
Codecov ReportBase: 88.59% // Head: 88.58% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #35029 +/- ##
===========================================
- Coverage 88.59% 88.58% -0.01%
===========================================
Files 2136 2136
Lines 396142 396150 +8
===========================================
- Hits 350948 350946 -2
- Misses 45194 45204 +10
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. |
This reverts commit 563b666.
About commit 0f486e3 : Oops my bad, I wanted to implement the depth of a quasimodular form, but accidently pushed the changes to this branch. |
Hello Vincent, I was wondering what do you have in mind when you say to refactor |
Indeed, |
Ok I see. I'll do the changes in a seperate PR. Also, there are some more refactoring and enhancement that could be done in the |
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.
Hey, you'll be receiving a few notifications from me, since I'm going through old PRs. Nevermind, there's only three PRs still open, and one I don't understand :)
I gave some feedback.
Hello, thank you for the reviews, I will address the comments in a few weeks. |
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 to me. Thanks!
Documentation preview for this PR (built with commit 4df388b; changes) is ready! 🎉 |
📚 Description
This PR implements the method
QuasiModularForms.basis_of_weight(k)
. It returns a list of quasimodular forms which generate the space of weight k. Some examples:CC: @videlec
📝 Checklist
⌛ Dependencies