Ticket #8876 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

Allows triangular morphisms to have domain and codomain with different index sets

Reported by: hivert Owned by: hivert
Priority: major Milestone: sage-4.4.4
Component: categories Keywords: Triangular Morphisms
Cc: Work issues:
Report Upstream: N/A Reviewers: Jason Bandlow
Authors: Florent Hivert Merged in: sage-4.4.4.alpha0
Dependencies: Stopgaps:

Description (last modified by hivert) (diff)

Currently triangular morphisms required to have domain and codomain with the same index sets. The patch remove this limitation by having TriangularMorphism accept a new argument inverse_on_support. It is supposed to be a function r : J |--> I returning the leading term i from the one of j = on_basis(i). This is used by: preimage and section.

I also wrote the methods co_kernel_projection, co_reduced, and _test_triangular.

Depend on #8881 and #8742, Apply patch in that order:

Attachments

trac_8876-triangular_morphisms_improve-fh.patch Download (27.1 KB) - added by hivert 3 years ago.
trac_8876-reviewer_patch-jb.patch Download (11.3 KB) - added by jbandlow 3 years ago.

Change History

Changed 3 years ago by hivert

comment:1 Changed 3 years ago by hivert

  • Status changed from new to needs_review
  • Description modified (diff)

Changed 3 years ago by jbandlow

comment:2 Changed 3 years ago by jbandlow

  • Reviewers set to Jason Bandlow

Hi Florent, I found some issues with the file as I was going through it, which I've fixed in the attached reviewer patch. (Many were probably my fault to begin with.) Everything else looks good to me. If you approve my reviewer patch and verify that all tests pass on massena without prereqs, you can set a positive review. (I will post my reviewer patch on the combinat queue immediately below yours right now.)

comment:3 Changed 3 years ago by hivert

  • Status changed from needs_review to positive_review
  • Description modified (diff)

Everything was ok with the following patch applied:

trac_8881-functorial_constructions-nt.patch
trac_8742-lazy_format-fh.patch
trac_8742-lazy_format-review-nt.patch
trac_8876-triangular_morphisms_improve-fh.patch
trac_8876-reviewer_patch-jb.patch

Thanks for the review and the corrections. Your patch is good. Positive review

comment:4 Changed 3 years ago by mhansen

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.4.4.alpha0
Note: See TracTickets for help on using tickets.