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

Implement SymmetricFunctions(QQ).inject_shorthands() #7777

Closed
nthiery opened this issue Dec 27, 2009 · 10 comments
Closed

Implement SymmetricFunctions(QQ).inject_shorthands() #7777

nthiery opened this issue Dec 27, 2009 · 10 comments

Comments

@nthiery
Copy link
Contributor

nthiery commented Dec 27, 2009

The title says it all. Depends on #7776

CC: @sagetrac-sage-combinat @jbandlow

Component: combinatorics

Keywords: symmetric functions, inject

Author: Nicolas M. Thiéry

Reviewer: Jason Bandlow

Merged: sage-4.3.1.rc0

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

@nthiery
Copy link
Contributor Author

nthiery commented Dec 27, 2009

comment:1

Beware: patch written and tested on Sage 4.2, not 4.3.

@jbandlow
Copy link

jbandlow commented Jan 7, 2010

comment:2

Although I haven't tested it yet, I'm in principle very happy with the 'green' part of the patch. Could you say a little about the 'red' part? What's being deleted from the 'introspect' files and why?

@nthiery
Copy link
Contributor Author

nthiery commented Jan 7, 2010

@jbandlow
Copy link

jbandlow commented Jan 9, 2010

comment:3

Looks good to me (on top of 7776). Thanks for this, Nicolas!

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 13, 2010

comment:4
The following tests failed:

        sage -t -long devel/sage-main/sage/combinat/sf/sf.py # 10 doctests failed

@rlmill rlmill mannequin added s: needs work and removed s: positive review labels Jan 13, 2010
@nthiery
Copy link
Contributor Author

nthiery commented Jan 13, 2010

comment:5

Replying to @rlmill:

The following tests failed:

        sage -t -long devel/sage-main/sage/combinat/sf/sf.py # 10 doctests failed

Thanks for the report. I did not manage yet to reproduce yet (I am downloading 4.3.1.alpha).
Could you send me a copy of the log? Was 7776 applied?

Cheers

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 13, 2010

comment:6

I had gotten too sleepy! Sorry, I didn't notice the dependency (it is 4am here now)...

@rlmill rlmill mannequin added s: needs review and removed s: needs work labels Jan 13, 2010
@nthiery
Copy link
Contributor Author

nthiery commented Jan 13, 2010

comment:8

Replying to @rlmill:

I had gotten too sleepy! Sorry, I didn't notice the dependency (it is 4am here now)...

:-)

We really should be using a ticket dependency plugin like:
http://trac-hacks.org/wiki/MasterTicketsPlugin.

Have a good night!

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 14, 2010

Merged: sage-4.3.1.rc0

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 14, 2010

Reviewer: Jason Bandlow

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