-
-
Notifications
You must be signed in to change notification settings - Fork 482
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
Improve genus method of function fields and curves #37538
Conversation
614bb3c
to
2ccffbf
Compare
Documentation preview for this PR (built with commit 3b31b1a; changes) is ready! 🎉 |
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.
Thank you. LGTM.
I think these are already nailed in memory in a few places (in one of the categories; if you want to import one from there, please go ahead). I think it would be better to have just one place where such things are constructed if we decide to change this in some fashion. It also can save loading time by a few milliseconds.
Thanks! |
📝 Checklist
⌛ Dependencies