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

remove deprecated stuff in padics #28663

Closed
fchapoton opened this issue Oct 28, 2019 · 13 comments
Closed

remove deprecated stuff in padics #28663

fchapoton opened this issue Oct 28, 2019 · 13 comments

Comments

@fchapoton
Copy link
Contributor

after #14825

CC: @xcaruso @roed314 @saraedum @tscrim

Component: refactoring

Author: Frédéric Chapoton

Branch/Commit: 3726356

Reviewer: Travis Scrimshaw, David Roe

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

@fchapoton fchapoton added this to the sage-9.0 milestone Oct 28, 2019
@fchapoton
Copy link
Contributor Author

Branch: u/chapoton/28663

@fchapoton
Copy link
Contributor Author

New commits:

bae385dremove deprecated stuff in padics

@fchapoton
Copy link
Contributor Author

Commit: bae385d

@fchapoton
Copy link
Contributor Author

comment:2

green bot, please review (easy one)

@tscrim
Copy link
Collaborator

tscrim commented Oct 28, 2019

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Oct 28, 2019

comment:3

LGTM.

@roed314
Copy link
Contributor

roed314 commented Oct 28, 2019

comment:4

I'm fine with these changes, but if we're going to remove the expansion deprecations for padic_ZZ_pX_CA_element we should really do it for the others too.

padics$ git grep "14825"
local_generic_element.pyx:            See http://trac.sagemath.org/14825 for details.
padic_ZZ_pX_CA_element.pyx:            deprecation(14825, "Interface to expansion has changed; first argument now n")
padic_ZZ_pX_CA_element.pyx:    list = deprecated_function_alias(14825, expansion)
padic_ZZ_pX_CA_element.pyx:    teichmuller_list = deprecated_function_alias(14825, teichmuller_expansion)
padic_ZZ_pX_CR_element.pyx:            deprecation(14825, "Interface to expansion has changed; first argument now n")
padic_ZZ_pX_CR_element.pyx:    list = deprecated_function_alias(14825, expansion)
padic_ZZ_pX_CR_element.pyx:    teichmuller_list = deprecated_function_alias(14825, teichmuller_expansion)
padic_ZZ_pX_FM_element.pyx:            deprecation(14825, "Interface to expansion has changed; first argument now n")
padic_ZZ_pX_FM_element.pyx:    list = deprecated_function_alias(14825, expansion)
padic_ZZ_pX_FM_element.pyx:    teichmuller_list = deprecated_function_alias(14825, teichmuller_expansion)
padic_generic_element.pyx:            See http://trac.sagemath.org/14825 for details.
padic_generic_element.pyx:        deprecation(14825, "__getitem__ is changing to match the behavior of number fields. Please use expansion instead.")
padic_template_element.pxi:            See http://trac.sagemath.org/14825 for details.
padic_template_element.pxi:        deprecation(14825, "list is deprecated. Please use expansion instead.")
padic_template_element.pxi:            See http://trac.sagemath.org/14825 for details.
padic_template_element.pxi:        deprecation(14825, "teichmuller_list is deprecated. Please use teichmuller_expansion instead.")
padic_template_element.pxi:            See http://trac.sagemath.org/14825 for details.
padic_template_element.pxi:        deprecation(14825, "padded_list is deprecated.  Please use expansion or Integer.digits with the padto keyword instead.")

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 28, 2019

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

3726356remove more deprecated stuff from 14825

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 28, 2019

Changed commit from bae385d to 3726356

@fchapoton
Copy link
Contributor Author

comment:6

ok. Done. I have left the ones concerning "getitem" aside, as they seem to warn about some future change, not yet in sage.

@roed314
Copy link
Contributor

roed314 commented Oct 28, 2019

comment:8

Yeah, I think leaving __getitem__ alone until we make the change mentioned there seems fine. Looks good to me.

@roed314
Copy link
Contributor

roed314 commented Oct 28, 2019

Changed reviewer from Travis Scrimshaw to Travis Scrimshaw, David Roe

@vbraun
Copy link
Member

vbraun commented Oct 29, 2019

Changed branch from u/chapoton/28663 to 3726356

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

4 participants