Skip to content
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

slicing p-adic elements #14106

Closed
roed314 opened this issue Feb 13, 2013 · 6 comments
Closed

slicing p-adic elements #14106

roed314 opened this issue Feb 13, 2013 · 6 comments

Comments

@roed314
Copy link
Contributor

roed314 commented Feb 13, 2013

sage: R = Zp(5,7)
sage: a = R(300)
sage: a
2*5^2 + 2*5^3 + O(5^9)
sage: a[:5]
4*5^2 + O(5^5)

This results from the following:

sage: a.slice(None, 5, 1) # correct
2*5^2 + 2*5^3 + O(5^5)
sage: a.slice(None, 5, None) # incorrect
4*5^2 + O(5^5)

CC: @saraedum

Component: padics

Reviewer: David Roe

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

@roed314 roed314 added this to the sage-5.9 milestone Feb 13, 2013
@roed314 roed314 self-assigned this Feb 13, 2013
@roed314
Copy link
Contributor Author

roed314 commented Feb 13, 2013

Attachment: 14106.patch.gz

@saraedum
Copy link
Member

comment:2

This is also done in #13299. (Which I should probably set to needs_review again). Should I make #13299 depend on #14106 or should we rather get #13299 into sage?

@saraedum
Copy link
Member

comment:4

I added the doctest from this patch to #13299 which is waiting for review.

@saraedum
Copy link
Member

comment:5

This is a part of #13299 now.

@saraedum saraedum removed this from the sage-5.9 milestone Mar 16, 2013
@jdemeyer
Copy link

Reviewer: David Roe

@jdemeyer
Copy link

Changed author from David Roe to none

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

No branches or pull requests

3 participants