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.

Sign up for GitHub

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

Jump to bottom

Fix a python 3 incompatibility using lambda #22359

Closed
mantepse opened this issue Feb 11, 2017 · 9 comments
Closed

Fix a python 3 incompatibility using lambda #22359

mantepse opened this issue Feb 11, 2017 · 9 comments
Labels
c: combinatorics p: major / 3 t: bug
Milestone
sage-7.6

Comments

@mantepse
Copy link
Collaborator

By ignorance, I used a tuple argument instead of 2 arguments. Fixed

CC: @fchapoton

Component: combinatorics

Author: Martin Rubey

Branch/Commit: 8b586f1

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/22359
@mantepse mantepse added this to the sage-7.6 milestone Feb 11, 2017
@mantepse
Copy link
Collaborator Author

Branch: u/mantepse/growth_diagram_python_3_fix

@mantepse
Copy link
Collaborator Author

Commit: 8b586f1

@mantepse
Copy link
Collaborator Author

comment:2

Frederic, I just managed to touch two of the things (I believe) you like :-)

New commits:

8b586f1replace tuple-argument lambda by 2-argument lambda

@mantepse

This comment has been minimized.

Sign in to view
@mantepse
Copy link
Collaborator Author

Author: Martin Rubey

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:3

lgtm, thanks

@mantepse
Copy link
Collaborator Author

comment:4

Thank you!

Could you also have a short look at #22356, which is a bit embarassing for me?

@vbraun
Copy link
Member

vbraun commented Feb 12, 2017

Changed branch from u/mantepse/growth_diagram_python_3_fix to 8b586f1

@vbraun vbraun closed this as completed in 39e2cdf Feb 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: combinatorics p: major / 3 t: bug
Projects
None yet
Milestone
sage-7.6
Development

No branches or pull requests
3 participants
@vbraun @fchapoton @mantepse