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

g.subs({x:1,y:2}) should walk through x,y sorted #6481

Closed
malb opened this issue Jul 8, 2009 · 2 comments
Closed

g.subs({x:1,y:2}) should walk through x,y sorted #6481

malb opened this issue Jul 8, 2009 · 2 comments

Comments

@malb
Copy link
Member

malb commented Jul 8, 2009

reported by Kwankyu on [sage-support]:

I mean the substitution y:x*y is applied first in the following

sage: R.<x,y>=QQ[]
sage: g=x+y
sage: g.subs({x:x+1,y:x*y})
x*y + x + y + 1

where I think applying x:x+1 first seems intuitive if order ever
should be significant.

Component: commutative algebra

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

@malb malb added this to the sage-4.1 milestone Jul 8, 2009
@malb malb self-assigned this Jul 8, 2009
@williamstein
Copy link
Contributor

comment:1

For the record, I think the entire design of subs for multivariate polynomial rings is wrong. I've thus opened #6482 and explained my reasoning for this.

Note that in any case, if the current subs behavior is super fast or useful to people (is it?) then we can keep it as a nondefault option, in which case this ticket #6481 also makes sense to keep, since at least we should do the order of substitution in an easy-to-understand way.

@rlmill rlmill mannequin modified the milestones: sage-4.1, sage-4.1.1 Jul 8, 2009
@malb
Copy link
Member Author

malb commented Sep 9, 2009

comment:3

Dupe of #6482

@malb malb closed this as completed Sep 9, 2009
@sagetrac-mvngu sagetrac-mvngu mannequin removed this from the sage-4.1.2 milestone Sep 20, 2009
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

2 participants