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
Better printing for the genus symbol #25776
Comments
Commit: |
New commits:
|
comment:4
LGTM (I don't understand the math, but the code matches what you put in the comments.). Once you add the author name, you can set a positive review. |
Reviewer: Travis Scrimshaw |
Author: Simon Brandhorst |
comment:6
I just noticed another bug:
Here Also it would be good to print the signature of the quadratic form and |
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
|
Dependencies: #25682 |
This comment has been minimized.
This comment has been minimized.
comment:11
Nitpicking: It would be a much nicer test output to run Otherwise LGTM (i.e., feel free to set a positive review if you don't want to change these things). |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:13
thank you. Those tricks were new to me. |
Changed branch from u/sbrandhorst/bug_in_printing_of_a_genus_symbol to |
The correct symbol is
Moreover the signature should be printed and
if the representative is very large it can be suppressed.
Depends on #25682
Component: quadratic forms
Keywords: genus
Author: Simon Brandhorst
Branch/Commit:
c979627
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/25776
The text was updated successfully, but these errors were encountered: