Opened 4 years ago

Closed 4 years ago

#22359 closed defect (fixed)

Fix a python 3 incompatibility using lambda

Reported by: mantepse Owned by:
Priority: major Milestone: sage-7.6
Component: combinatorics Keywords:
Cc: chapoton Merged in:
Authors: Martin Rubey Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 8b586f1 (Commits, GitHub, GitLab) Commit: 8b586f1ab333fc075a76b6abcd37d0eaaf83560f
Dependencies: Stopgaps:

Status badges

Description (last modified by mantepse)

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

Change History (5)

comment:1 Changed 4 years ago by mantepse

  • Branch set to u/mantepse/growth_diagram_python_3_fix

comment:2 Changed 4 years ago by mantepse

  • Authors set to Martin Rubey
  • Cc chapoton added
  • Commit set to 8b586f1ab333fc075a76b6abcd37d0eaaf83560f
  • Component changed from PLEASE CHANGE to combinatorics
  • Description modified (diff)
  • Status changed from new to needs_review
  • Type changed from PLEASE CHANGE to defect

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

New commits:

8b586f1replace tuple-argument lambda by 2-argument lambda

comment:3 Changed 4 years ago by chapoton

  • Reviewers set to Frédéric Chapoton
  • Status changed from needs_review to positive_review

lgtm, thanks

comment:4 Changed 4 years ago by mantepse

Thank you!

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

comment:5 Changed 4 years ago by vbraun

  • Branch changed from u/mantepse/growth_diagram_python_3_fix to 8b586f1ab333fc075a76b6abcd37d0eaaf83560f
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.