Opened 11 years ago

Last modified 9 years ago

#12900 closed enhancement

(Demazure) roots of toric varieties — at Version 9

Reported by: Volker Braun Owned by: Alex Ghitza
Priority: major Milestone: sage-5.12
Component: algebraic geometry Keywords:
Cc: Andrey Novoseltsev Merged in:
Authors: Volker Braun Reviewers: Andrey Novoseltsev, Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13023 Stopgaps:

Status badges

Description (last modified by Volker Braun)

The Demazure roots (often just called roots) of a toric variety give you the non-Abelian part of the continuous automorphisms.

Apply

Change History (9)

comment:1 Changed 11 years ago by Volker Braun

Cc: Andrey Novoseltsev added
Dependencies: #12892
Status: newneeds_review

Doesn't use anything from #12892, but otherwise this ticket applies with fuzz.

comment:2 Changed 11 years ago by Andrey Novoseltsev

Reviewers: Andrey Novoseltsev

Should there perhaps be a check that the variety is complete? Also would be nice to have a direct link to arXiv.

comment:3 Changed 11 years ago by Volker Braun

For incomplete varieties it would already bomb out because it can't enumerate the points in the (non-compact) dual polytope. But its better to have an explicit check with meaningful error message. Also fixed the typesetting of the doctest.

comment:4 Changed 11 years ago by Volker Braun

Dependencies: #12892#12892, #13023

I've updated the paths for #13023, and checked that all doctests pass.

comment:5 Changed 9 years ago by Frédéric Chapoton

looks good to me.

I propose to remove the dependency to #12892, which is not really needed.

If you agree, you can remove the dependency and set a positive review.

comment:6 Changed 9 years ago by Volker Braun

Dependencies: #12892, #13023#13023
Reviewers: Andrey NovoseltsevAndrey Novoseltsev, Frédéric Chapoton
Status: needs_reviewpositive_review

I've rebased the original patch to get rid off fuzz.

comment:7 Changed 9 years ago by Jeroen Demeyer

Milestone: sage-5.11sage-5.12

comment:8 Changed 9 years ago by Jeroen Demeyer

Status: positive_reviewneeds_info

Please clarify which patch(es) should be applied.

comment:9 Changed 9 years ago by Volker Braun

Description: modified (diff)
Status: needs_infopositive_review

patchbot: apply trac_12900_Demazure_roots.patch, trac_12900_review.patch

Note: See TracTickets for help on using tickets.