Opened 6 months ago

Closed 5 months ago

Last modified 4 months ago

#30704 closed enhancement (fixed)

Upgrade to Normaliz 3.8.9 and PyNormaliz 2.13

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.3
Component: packages: optional Keywords:
Cc: Winfried, gh-kliem, jipilab Merged in:
Authors: Matthias Koeppe, Jonathan Kliem Reviewers: Jonathan Kliem, Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: 0e8f99d (Commits, GitHub, GitLab) Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by mkoeppe)

Release notes:

Tarballs: See checksums.ini

Change History (15)

comment:1 Changed 6 months ago by mkoeppe

  • Branch set to u/mkoeppe/upgrade_to_normaliz_3_8_9_and_pynormaliz_2_13

comment:2 Changed 6 months ago by mkoeppe

  • Commit set to 02386d4f01e43ff9ab9a0e87bfd194dc9b3e3363
  • Description modified (diff)

New commits:

​91eb419build/pkgs/normaliz: Upgrade to 3.8.9
​02386d4build/pkgs/pynormaliz: Upgrade to 2.13

comment:3 Changed 6 months ago by mkoeppe

  • Authors set to Matthias Koeppe, ...

comment:4 Changed 6 months ago by mkoeppe

Todo (from #30531): Use new matrix TriangulationGenerators that stores the ordered set of generators used in building the triangulation

comment:5 Changed 6 months ago by gh-kliem

  • Authors changed from Matthias Koeppe, ... to Matthias Koeppe, Jonathan Kliem
  • Branch changed from u/mkoeppe/upgrade_to_normaliz_3_8_9_and_pynormaliz_2_13 to u/gh-kliem/upgrade_to_normaliz_3_8_9_and_pynormaliz_2_13
  • Commit changed from 02386d4f01e43ff9ab9a0e87bfd194dc9b3e3363 to 0e8f99d581dbde532bf6b4522401dc7978effb6e
  • Status changed from new to needs_review

New commits:

​211e9c5Merge branch 'u/mkoeppe/upgrade_to_normaliz_3_8_9_and_pynormaliz_2_13' of git://trac.sagemath.org/sage into u/kliem/upgrade_to_normaliz_3_8_9_and_pynormaliz_2_13
​0e8f99duse TriangulationGenerators

comment:6 Changed 6 months ago by mkoeppe

  • Milestone changed from sage-9.2 to sage-9.3

comment:7 Changed 5 months ago by gh-kliem

Are we waiting on something yet?

comment:8 Changed 5 months ago by mkoeppe

  • Reviewers set to ..., Matthias Koeppe

Looks good to me

comment:9 Changed 5 months ago by gh-kliem

  • Reviewers changed from ..., Matthias Koeppe to Jonathan Kliem, Matthias Koeppe
  • Status changed from needs_review to positive_review

comment:10 Changed 5 months ago by vbraun

  • Branch changed from u/gh-kliem/upgrade_to_normaliz_3_8_9_and_pynormaliz_2_13 to 0e8f99d581dbde532bf6b4522401dc7978effb6e
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:11 Changed 4 months ago by jipilab

  • Commit 0e8f99d581dbde532bf6b4522401dc7978effb6e deleted

Tarball of pynormaliz 2.13 seems to be missing on the mirrors (Sorry if you already know, or if it is a question of time)

comment:12 Changed 4 months ago by tscrim

It is worse that: the checksum of the tarball from the link doesn't match this ticket.

comment:13 Changed 4 months ago by gh-kliem

Can you be more precices? Which checksum doesn't match?

If I download ​https://pypi.io/packages/source/p/pynormaliz/PyNormaliz-2.13.tar.gz

I get the same checksums as in this ticket.

comment:14 Changed 4 months ago by gh-kliem

I see, it doesn't match ​https://github.com/Normaliz/PyNormaliz/releases/tag/v2.13.

I don't know, why this would be different.

comment:15 Changed 4 months ago by tscrim

That probably is the issue as Volker likely pulled from the link to the mirrors.

Note: See TracTickets for help on using tickets.