Ticket #5729 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] Cleanup of crystal code: cartan_type now a method rather than attribute

Reported by: aschilling Owned by: aschillin
Priority: major Milestone: sage-3.4.1
Component: combinatorics Keywords:
Cc: sage-combinat Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description (last modified by nthiery) (diff)

Crystals: cartan_type is a method

  • Changed the API to have cartan type as a method rather than attribute
  • Systematically use .parent() instead of ._parent
  • Minor doc improvements

Attachments

crystal-cleanup-track.patch Download (18.1 KB) - added by aschilling 4 years ago.
crystal-5729-track.patch Download (27.4 KB) - added by aschilling 4 years ago.
changed according to Nicolas' suggestions
crystal-5729-track.2.patch Download (30.7 KB) - added by nthiery 4 years ago.

Change History

Changed 4 years ago by aschilling

Changed 4 years ago by aschilling

changed according to Nicolas' suggestions

comment:1 Changed 4 years ago by aschilling

  • Summary changed from Cleanup of crystal code: cartan_type now a method rather than attribute to [with patch, needs review] Cleanup of crystal code: cartan_type now a method rather than attribute

Changed 4 years ago by nthiery

comment:2 Changed 4 years ago by nthiery

  • Cc sage-combinat added
  • Summary changed from [with patch, needs review] Cleanup of crystal code: cartan_type now a method rather than attribute to [with patch, positive review] Cleanup of crystal code: cartan_type now a method rather than attribute

Final version of the patch uploaded:

  • fixes 2/3 remaining calls to parent()
  • Micro doc improvements

comment:3 Changed 4 years ago by nthiery

  • Description modified (diff)

comment:4 Changed 4 years ago by aschilling

  • Status changed from new to closed
  • Resolution set to fixed

comment:5 Changed 4 years ago by mabshoff

  • Status changed from closed to reopened
  • Resolution fixed deleted

Huh? This ticket has not been merged.

Cheers,

Michael

comment:6 Changed 4 years ago by mabshoff

  • Status changed from reopened to closed
  • Resolution set to fixed

Merged crystal-5729-track.2.patch in Sage 3.4.1.rc2.

Cheers,

Michael

Note: See TracTickets for help on using tickets.