Changes between Initial Version and Version 7 of Ticket #14353


Ignore:
Timestamp:
04/10/13 05:14:16 (9 years ago)
Author:
novoselt
Comment:

Hi Volker, I propose this version based on triple factorization for both fan and variety morphisms. I hope the documentation is written clearly and the example chosen gives a good idea what is what. The only remaining problem is that I really want to multiply factors back into the original one. For fan morphisms I just added __mul__ as they are standalone (although it may make sense to change that...), but toric morphisms should follow coercion rules, so implementing the double underscore method as it is done now is not acceptable. And even with it there is an issue of comparing, although it does seem that the result is correct.

Anyway: let me know what you think of composition and if the rest looks OK!

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14353

    • Property Status changed from new to needs_work
    • Property Work issues changed from to morphism composition
    • Property Authors changed from Volker Braun to Volker Braun, Andrey Novoseltsev
    • Property Cc novoselt added
    • Property Keywords toric added
    • Property Reviewers changed from to Andrey Novoseltsev, Volker Braun
  • Ticket #14353 – Description

    initial v7  
    1 This ticket implements the HLY factorization into a surjective and a generically injective toric morphism.
     1This ticket implements factorization into a surjective, birational, and injective toric morphism.
     2
     3'''Apply:'''
     4 1. [attachment:trac_14353_fan_morphism_factoring.patch]
     5 1. [attachment:trac_14353_auxiliary_methods.patch]
     6 1. [attachment:trac_14353_toric_morphism_factoring.patch]