We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
In general, N.sqrt(extend=False, all=True) should never raise an exception if N is not a square. Instead, it should just return an empty list.
N.sqrt(extend=False, all=True)
N
We also fix 0.sqrt(prec=53) which should return a real instead of an integer.
0.sqrt(prec=53)
CC: @tscrim
Component: basic arithmetic
Author: Jeroen Demeyer
Branch/Commit: 880b0b5
880b0b5
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/26509
The text was updated successfully, but these errors were encountered:
Branch: u/jdemeyer/__1__sqrt_extend_false__all_true__raises_an_exception
Sorry, something went wrong.
Commit: 880b0b5
New commits:
Various fixes to Integer.sqrt()
LGTM.
Changed branch from u/jdemeyer/__1__sqrt_extend_false__all_true__raises_an_exception to 880b0b5
2d41cc8
lift_x()
is_x_coord()
No branches or pull requests
In general,
N.sqrt(extend=False, all=True)
should never raise an exception ifN
is not a square. Instead, it should just return an empty list.We also fix
0.sqrt(prec=53)
which should return a real instead of an integer.CC: @tscrim
Component: basic arithmetic
Author: Jeroen Demeyer
Branch/Commit:
880b0b5
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/26509
The text was updated successfully, but these errors were encountered: