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

injvar() docstring should be the same as inject_variables() #4143

Closed
haraldschilly opened this issue Sep 18, 2008 · 4 comments
Closed

injvar() docstring should be the same as inject_variables() #4143

haraldschilly opened this issue Sep 18, 2008 · 4 comments

Comments

@haraldschilly
Copy link
Member

The injvar() command has no docstring. Maybe depreciate it and use the docstring of inject_variables() ?

R = PolynomialRing( GF(Integer(2)), ['a%s'%i for i in range(Integer(93))] + ['b%s'%i for i in range(Integer(84))], order='degrevlex' )

R.injvar?
Type:           builtin_function_or_method
Base Class:     <type 'builtin_function_or_method'>
String Form:    <built-in method injvar of sage.rings.polynomial.multi_polynomial_libsingular.MPolynomialRing_libsingular object at 0xb1c32414>
Namespace:      Interactive
Docstring:
    <no docstring>
Class Docstring:
    <attribute '__doc__' of 'builtin_function_or_method' objects>

Component: algebra

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

@malb
Copy link
Member

malb commented Sep 18, 2008

comment:1

There should be one-- and preferably only one --obvious way to do it.

(http://www.python.org/dev/peps/pep-0020/)

I vote for deprecation of injvar.

@jhpalmieri
Copy link
Member

comment:2

Attachment: 4143.patch.gz

This patch deprecates injvar.

@robertwb
Copy link
Contributor

comment:3

I agree with deprecating this, and the patch looks good to me.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Dec 12, 2008

comment:4

Merged in Sage 3.2.2.alpha2

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