Skip to content

Conversation

@gulshan-123
Copy link

@gulshan-123 gulshan-123 commented Dec 17, 2025

📝 Checklist

In #41276, I added the printing mode='digits-unicode' in p-adic printing.
This addition was to avoid ... being treated as wildcard in pydoctest module. digits-unicode replaced ... with unicode

This PR updates the doc to solve the problem.
I updated the docstring as well as the doctest to reflect the new change.

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

#41276: This branch is after the branch in previous PR,, thus must close previous PR first..


TODO:
Update in more files: factory.py and generic_nodes.py

@user202729
Copy link
Contributor

you mean to link to #41276 instead of #41270 .

sage: repr(Qp(17)(100))
'...5F'
sage: repr(Qp(17)(1000))
'...37E'
Copy link
Contributor

@user202729 user202729 Dec 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

leave this one test alone. We need something to at least ensure mode('digits') doesn't error out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants