#26479 closed defect (fixed)
padic printer does not support latex names
Reported by: | caruso | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-8.5 |
Component: | padics | Keywords: | |
Cc: | roed | Merged in: | |
Authors: | Xavier Caruso | Reviewers: | David Roe, Frédéric Chapoton |
Report Upstream: | N/A | Work issues: | |
Branch: | fb3bbaa (Commits, GitHub, GitLab) | Commit: | fb3bbaa1e333112e4ee34bec5712b45a845841ac |
Dependencies: | Stopgaps: |
Description
sage: K.<pi> = Qp(2).extension(x^3 - 2) sage: latex(pi) 'pi + O(pi^61)'
Change History (11)
comment:1 Changed 3 years ago by
- Branch set to u/caruso/padic_printer_latex
- Commit set to 8b1816472114ac456a509e11cdd974274041fe4b
- Status changed from new to needs_review
comment:2 follow-up: ↓ 4 Changed 2 years ago by
comment:3 Changed 2 years ago by
- Commit changed from 8b1816472114ac456a509e11cdd974274041fe4b to 008d7fc45c055246740f55d524fca084f19914e0
comment:4 in reply to: ↑ 2 Changed 2 years ago by
Replying to chapoton:
Please undo all changes from
\ZZ
to\Bold{Z}
in the documentation.For the uniformity of sage documentation, it is better to always use
\ZZ
.
Ah OK. It's a kind of trap because we cannot use \ZZ
in the notebook for instance.
Anyway, I made the changes and also replaced \mathbb{Z}
by \ZZ
, etc. Please tell me if it's not appropriate.
comment:5 Changed 2 years ago by
I think we also have \GF
for finite fields. But all these cosmetic changes should really be done IMHO in another ticket, because they currently hide the important changes in this ticket, which get lost in the middle of all the changes made.
comment:6 Changed 2 years ago by
- Commit changed from 008d7fc45c055246740f55d524fca084f19914e0 to 9e85f914c9bbfbb45c0123b241512ffba24bb757
comment:7 Changed 2 years ago by
OK, if you insist, I split this ticket into two parts.
comment:8 Changed 2 years ago by
- Commit changed from 9e85f914c9bbfbb45c0123b241512ffba24bb757 to fb3bbaa1e333112e4ee34bec5712b45a845841ac
Branch pushed to git repo; I updated commit sha1. New commits:
fb3bbaa | Fix doctests
|
comment:9 Changed 2 years ago by
- Reviewers set to David Roe, Frédéric Chapoton
- Status changed from needs_review to positive_review
Looks good! I ran long tests and they pass.
comment:10 Changed 2 years ago by
- Branch changed from u/caruso/padic_printer_latex to fb3bbaa1e333112e4ee34bec5712b45a845841ac
- Resolution set to fixed
- Status changed from positive_review to closed
comment:11 Changed 2 years ago by
- Milestone changed from sage-8.4 to sage-8.5
This should be re-targeted for 8.5.
Please undo all changes from
\ZZ
to\Bold{Z}
in the documentation.For the uniformity of sage documentation, it is better to always use
\ZZ
.