padic printer does not support latex names

[xcaruso]

sage: K.<pi> = Qp(2).extension(x^3 - 2)
sage: latex(pi)
'pi + O(pi^61)'

CC: @roed314

Component: padics

Author: Xavier Caruso

Branch/Commit: fb3bbaa

Reviewer: David Roe, Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/26479

[xcaruso]

Branch: u/caruso/padic_printer_latex

[xcaruso]

Commit: 8b18164

[fchapoton]

comment:2

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.

[sagetrac-git]

Changed commit from 8b18164 to 008d7fc

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

5ca2bd0	Replace back \Bold{Z} by \ZZ, etc. in the documentation
83b48e3	Replace \mathbb{Z}, \mathbb{Q} by \ZZ, \QQ respectively
008d7fc	Replace \mathbb{f} by \Bold{F}

[xcaruso]

comment:4

Replying to @fchapoton:

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.

[fchapoton]

comment:5

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.

[sagetrac-git]

Changed commit from 008d7fc to 9e85f91

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

e83713c	Revert "Replace \mathbb{f} by \Bold{F}"
2e41efd	Revert "Replace \mathbb{Z}, \mathbb{Q} by \ZZ, \QQ respectively"
9e85f91	Small fixes

[xcaruso]

comment:7

OK, if you insist, I split this ticket into two parts.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

fb3bbaa

Fix doctests

[sagetrac-git]

Changed commit from 9e85f91 to fb3bbaa

[roed314]

comment:9

Looks good! I ran long tests and they pass.

[roed314]

Reviewer: David Roe, Frédéric Chapoton

[roed314]

Author: Xavier Caruso

[vbraun]

Changed branch from u/caruso/padic_printer_latex to fb3bbaa

[embray]

comment:11

This should be re-targeted for 8.5.