Fixing unexpected behaviour in ToricHigherDirectImages #3884
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was unexpected behaviour from the output of HDI, which ultimately came from an incorrect lattice point computation in a polyhedron. I added an internal method for the correct computation
idealFromPolyhedron
which usesrawHilbertBasis
to compute the appropriate lattice points. I also added the example which demonstrated the incorrect behaviour as a test.The example in question is
The source is an interated blowup of PP^2, and this is computing the pushforward of a line bundle via the composition of blowdowns, a birational map. The output turns out to be of rank two, which doesn't make sense---the fibres should generically be rank one. The corrected version appropriately outputs a presentation of the ideal m^3, where m is the blown up point.