Closed jplab closed 5 years ago
Branch: u/jipilab/normaliz_features
Dependencies: #22984, #25090
Last 10 new commits:
737552d | Updating patch with upstream fix for wrong number of lattice points. |
530302d | Merge branch 'u/tscrim/upgrade_noramliz_pynormaliz-22984' of git://trac.sagemath.org/sage into u/tscrim/upgrade_noramliz_pynormaliz-22984 |
57d40b0 | Upgrade Normaliz to 3.5.2. |
6ba442d | Adding tests from [comment:24](#comment%3A24),25 of #22984. |
183087a | Upgrade PyNormaliz to 1.12 |
7f47ec9 | Adapted the polyhedron docstring |
117e428 | Upgrade normaliz to 3.5.3 |
7057ef4 | Merge branch to get docstring adaptation |
1b37e8b | Merge branch 'develop' into test_normaliz |
e421ff8 | First version of integral pts gen |
Changed branch from u/jipilab/normaliz_features to u/jipilab/some_normaliz_features
Changed branch from u/jipilab/some_normaliz_features to public/some_normaliz_features
Branch pushed to git repo; I updated commit sha1. New commits:
3d3f64a | Handle default behavior |
Note that is_package_installed
is being deprecated, see #20382
Branch pushed to git repo; I updated commit sha1. New commits:
55c9a04 | Added Hilbert series |
Branch pushed to git repo; I updated commit sha1. New commits:
720dc04 | Made tests pass |
Branch pushed to git repo; I updated commit sha1. New commits:
a4ddeac | Cropped feature methods |
Branch pushed to git repo; I updated commit sha1. New commits:
e32c9fd | Added triangulate with normaliz |
Description changed:
---
+++
@@ -6,3 +6,5 @@
- Triangulations
This ticket provides some new features.
+
+TO DO: Make the bridge with PointConfiguration `triangulate` and the triangulation via normaliz.
Branch pushed to git repo; I updated commit sha1. New commits:
45c39b8 | Some small edits |
Branch pushed to git repo; I updated commit sha1. New commits:
5679821 | Added handling the triangulation with point configurations |
Branch pushed to git repo; I updated commit sha1. New commits:
b045caf | Adapted inhomogeneous case |
Description changed:
---
+++
@@ -3,8 +3,5 @@
- Integral points generators
- Volume
- Euclidean volume
-- Triangulations
-
-This ticket provides some new features.
-
-TO DO: Make the bridge with PointConfiguration `triangulate` and the triangulation via normaliz.
+- Triangulation
+- Hilbert series
It's probably best to add all lattice-point related methods to Polyhedron_QQ_normaliz
rather than Polyhedron_normaliz
in anticipation of #25097 (which moves integral_points
and integral_hull
there).
Replying to @mkoeppe:
It's probably best to add all lattice-point related methods to
Polyhedron_QQ_normaliz
rather thanPolyhedron_normaliz
in anticipation of #25097 (which movesintegral_points
andintegral_hull
there).
I see. Yes I can do that.
Replying to @jplab:
Replying to @mkoeppe:
It's probably best to add all lattice-point related methods to
Polyhedron_QQ_normaliz
rather thanPolyhedron_normaliz
in anticipation of #25097 (which movesintegral_points
andintegral_hull
there).I see. Yes I can do that.
As discussed with mkoeppe: The move of the rational polyhedron methods will be done in #25097. The merge should go without conflicts.
Replying to @videlec:
Note that
is_package_installed
is being deprecated, see #20382
Oh! Wow! Thanks for the timely warning! I set it as a dependency. I will change the check once it is merged...
Description changed:
---
+++
@@ -5,3 +5,5 @@
- Euclidean volume
- Triangulation
- Hilbert series
+
+TODO: Adapt the `is_package_installed` for the volume part (see #20382).
Changed dependencies from #22984, #25090 to #22984, #25090, #20382
Branch pushed to git repo; I updated commit sha1. New commits:
e7be14a | Fixed some polytopes and added some tests |
Branch pushed to git repo; I updated commit sha1. New commits:
d0e2498 | Added the Ehrhart Series and Quasi-Polynomial |
Branch pushed to git repo; I updated commit sha1. New commits:
668fdfc | Added an example |
Using Normaliz can compute several things that are currently not interfaced. This ticket implements:
FOLLOWUP:
.volume()
that can have several engines.Polyhedra_normaliz_QQ
.Depends on #27682 Depends on #27716
CC: @videlec @mo271 @mkoeppe @sebasguts @w-bruns @braunmath @tscrim
Component: geometry
Keywords: polytope, normaliz, IMA-PolyGeom
Author: Jean-Philippe Labbé, Matthias Koeppe
Branch/Commit:
da05706
Reviewer: Travis Scrimshaw, Jeroen Demeyer
Issue created by migration from https://trac.sagemath.org/ticket/25091