flatsurf / surface-dynamics

Teichmüller dynamics in SageMath
https://flatsurf.github.io/surface-dynamics/
GNU General Public License v2.0
12 stars 14 forks source link

PyPI release is outdated #81

Closed mkoeppe closed 8 months ago

mkoeppe commented 9 months ago

In Sage, make sage_flatsurf fails.

make --no-print-directory surface_dynamics-no-deps
sage-logger -p 'sage --pip install -r "/Users/mkoeppe/s/sage/sage-rebasing/worktree-green/build/pkgs/surface_dynamics/requirements.txt"' '/Users/mkoeppe/s/sage/sage-rebasing/worktree-green/logs/pkgs/surface_dynamics.log'
[surface_dynamics] Collecting surface_dynamics (from -r /Users/mkoeppe/s/sage/sage-rebasing/worktree-green/build/pkgs/surface_dynamics/requirements.txt (line 1))
[surface_dynamics]   Downloading surface_dynamics-0.4.7.tar.gz (8.3 MB)
[surface_dynamics]      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 8.3/8.3 MB 5.3 MB/s eta 0:00:00
[surface_dynamics]   Preparing metadata (setup.py): started
[surface_dynamics]   Preparing metadata (setup.py): finished with status 'error'
[surface_dynamics]   error: subprocess-exited-with-error
[surface_dynamics]   
[surface_dynamics]   × python setup.py egg_info did not run successfully.
[surface_dynamics]   │ exit code: 1
[surface_dynamics]   ╰─> [47 lines of output]
[surface_dynamics]       /Users/mkoeppe/s/sage/sage-rebasing/worktree-green/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/Cython/Compiler/Main.py:381: FutureWarning: Cython directive 'language_level' not set, using '3str' for now (Py3). This has changed from earlier releases! File: /private/var/folders/38/wnh4gf1552g_crsjnv2vmmww0000gp/T/pip-install-yatjhv32/surface-dynamics_99b12edf7e9b4c4e8b650fccc5e60ca1/surface_dynamics/flat_surfaces/origamis/origami_dense.pxd
[surface_dynamics]         tree = Parsing.p_module(s, pxd, full_module_name)
[surface_dynamics]       /Users/mkoeppe/s/sage/sage-rebasing/worktree-green/local/var/lib/sage/venv-python3.11/lib/python3.11/site-packages/Cython/Compiler/Main.py:381: FutureWarning: Cython directive 'language_level' not set, using '3str' for now (Py3). This has changed from earlier releases! File: /private/var/folders/38/wnh4gf1552g_crsjnv2vmmww0000gp/T/pip-install-yatjhv32/surface-dynamics_99b12edf7e9b4c4e8b650fccc5e60ca1/surface_dynamics/interval_exchanges/iet_family.pyx
[surface_dynamics]         tree = Parsing.p_module(s, pxd, full_module_name)
[surface_dynamics]       
[surface_dynamics]       Error compiling Cython file:
[surface_dynamics]       ------------------------------------------------------------
[surface_dynamics]       ...
[surface_dynamics]                           mpz_set(self.rows[self.length][j],
[surface_dynamics]                                   deref(gsi_ptr[0]).coefficient(PPL_Variable(j)).get_mpz_t())
[surface_dynamics]                       self.length += 1
[surface_dynamics]                   gsi_ptr[0].inc(1)
[surface_dynamics]               del gsi_ptr
[surface_dynamics]               qsort(self.rows, self.length, sizeof(mpz_t *), & compare)
[surface_dynamics]                                                              ^
[surface_dynamics]       ------------------------------------------------------------
[surface_dynamics]       

as noted in https://github.com/sagemath/sage/wiki/Sage-10.2-Release-Tour#known-problems-and-workarounds

saraedum commented 9 months ago

@videlec any objections to creating a 0.5.0 release from your side?

mkoeppe commented 9 months ago

@saraedum also please see #82

videlec commented 9 months ago

@videlec any objections to creating a 0.5.0 release from your side?

Not at all. It is fine with me.