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
Simpler caching in ell_rational_field #23970
Milestone
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Instead of using a
dict
to cache results depending onproof
, use a tuple(answer, proven)
. This is easier to understand and results in simpler code. This was already done for__rank
in #23962.Depends on #23962
CC: @koffie
Component: elliptic curves
Author: Jeroen Demeyer
Branch/Commit:
8bce6ff
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/23970
The text was updated successfully, but these errors were encountered: