Skip to content

Add digits-unicode option to print_mode in p-adic numbers #41276

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.

Sign up for GitHub

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

Open
gulshan-123 wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Add digits-unicode option to print_mode in p-adic numbers #41276

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
79480ae
digits or digits-unicode + replaced ellipsis
gulshan-123 Dec 8, 2025
a3f6310
doctest added: note: in digits doctest, ... is treated as wildcard
gulshan-123 Dec 8, 2025
652ed82
Merge branch 'develop' into digits-unicode_padic
gulshan-123 Dec 16, 2025
9c033b5
reordered tests
gulshan-123 Dec 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/sage/rings/padics/factory.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -283,9 +283,9 @@ def get_key_base(p, prec, type, print_mode, names, ram_name, print_pos, print_se
print_max_terms = padic_printing._printer_defaults.max_series_terms()

# We eliminate irrelevant print options (e.g. print_pos if p = 2)
if p == 2 or print_mode == 'digits':
if p == 2 or print_mode in {'digits', 'digits-unicode'}:
print_pos = True # we want this hard-coded so that we don't get duplicate parents if the keys differ.
if print_mode == 'digits':
if print_mode in {'digits', 'digits-unicode'}:
print_ram_name = None
print_alphabet = print_alphabet[:p]
else:
Expand Down
32 changes: 22 additions & 10 deletions src/sage/rings/padics/padic_printing.pyx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ cdef enum print_modes:
val_unit
digits
bars
digits_unicode


def pAdicPrinter(ring, options={}):
Expand Down Expand Up @@ -138,6 +139,9 @@ class pAdicPrinterDefaults(SageObject):
'...5F'
sage: repr(Qp(17)(1000))
'...37E'
sage: padic_printing.mode('digits-unicode')
sage: repr(Qp(17)(100))
'…0000000000000000005F'
sage: padic_printing.mode('bars')
sage: repr(Qp(19)(1000))
'...2|14|12'
Expand All @@ -147,7 +151,7 @@ class pAdicPrinterDefaults(SageObject):
if mode is None:
return self._mode

if mode in ['val-unit', 'series', 'terse', 'digits', 'bars']:
if mode in ['val-unit', 'series', 'terse', 'digits', 'bars', 'digits-unicode']:
self._mode = mode
else:
raise ValueError("invalid printing mode")
Expand Down Expand Up @@ -401,11 +405,14 @@ cdef class pAdicPrinter_class(SageObject):
self.mode = series
elif mode == 'terse':
self.mode = terse
elif mode == 'digits':
elif mode in {'digits', 'digits-unicode'}:
if len(self.alphabet) < self.prime_pow.prime or (not self.base and ring.absolute_f() != 1):
raise ValueError("digits printing mode only usable for totally ramified extensions with p at most the length of the alphabet (default 62). Try using print_mode = 'bars' instead.")
else:
self.mode = digits
if mode == 'digits-unicode':
self.mode = digits_unicode
else:
self.mode = digits
self.pos = True
elif mode == 'bars':
self.mode = bars
Expand Down Expand Up @@ -546,7 +553,7 @@ cdef class pAdicPrinter_class(SageObject):
if lx != rx:
return richcmp_not_equal(lx, rx, op)

if self.mode != digits:
if self.mode != digits and self.mode != digits_unicode:
lx = self.ram_name
rx = other.ram_name
if lx != rx:
Expand All @@ -558,13 +565,13 @@ cdef class pAdicPrinter_class(SageObject):
if lx != rx:
return richcmp_not_equal(lx, rx, op)

if self.mode == digits:
if self.mode == digits or self.mode == digits_unicode:
lx = self.alphabet[:p]
rx = other.alphabet[:q]
if lx != rx:
return richcmp_not_equal(lx, rx, op)

if self.mode == series or self.mode == digits or self.mode == bars:
if self.mode == series or self.mode == digits or self.mode == digits_unicode or self.mode == bars:
lx = self.max_ram_terms
rx = other.max_ram_terms
if lx != rx:
Expand Down Expand Up @@ -781,6 +788,8 @@ cdef class pAdicPrinter_class(SageObject):
return 'terse'
elif self.mode == digits:
return 'digits'
elif self.mode == digits_unicode:
return 'digits-unicode'
elif self.mode == bars:
return 'bars'

Expand Down Expand Up @@ -869,7 +878,7 @@ cdef class pAdicPrinter_class(SageObject):
elif mode == 'bars':
_mode = bars
else:
raise ValueError("printing mode must be one of 'val-unit', 'series', 'terse', 'bars', or 'digits'")
raise ValueError("printing mode must be one of 'val-unit', 'series', 'terse', 'bars', 'digits' or 'digits-unicode'")
if pos is None:
_pos = self.pos
else:
Expand Down Expand Up @@ -929,7 +938,7 @@ cdef class pAdicPrinter_class(SageObject):
return "0"
elif elt._is_inexact_zero():
prec = elt.precision_absolute()
if mode == digits and self.show_prec == "dots":
if (mode == digits or mode == digits_unicode) and self.show_prec == "dots":
if prec > 0:
s = (self.alphabet[0] * prec)
else:
Expand Down Expand Up @@ -960,7 +969,7 @@ cdef class pAdicPrinter_class(SageObject):
s = "%s * %s" % (ram_name, self._repr_spec(elt.unit_part(), do_latex, pos, terse, 1, ram_name))
else:
s = "%s^%s * %s" % (ram_name, elt.valuation(), self._repr_spec(elt.unit_part(), do_latex, pos, terse, 1, ram_name))
elif mode == digits:
elif mode == digits or mode == digits_unicode:
n = elt.valuation()
if self.base:
L = self.base_p_list(elt, True)
Expand Down Expand Up @@ -1030,7 +1039,10 @@ cdef class pAdicPrinter_class(SageObject):
s = self._repr_spec(elt, do_latex, pos, mode, 0, ram_name)

if self.show_prec == "dots":
s = "..." + s
if mode == digits_unicode:
s = u"\u2026" + s
else:
s = "..." + s
if self.show_prec == "bigoh":
if s == "":
s = "O(%s" % ram_name
Expand Down
Loading