search

conda-forge / pynormaliz-feedstock

A conda-smithy repository for pynormaliz.
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

Pin on normaliz should be stricter #10

Open saraedum opened 7 months ago

saraedum commented 7 months ago

Solution to issue cannot be found in the documentation.

Issue

Currently, pynormaliz is not functional with the latest normaliz.

>>> import PyNormaliz
[1]    7399 segmentation fault (core dumped)  python

Installed packages

_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                       2_gnu    conda-forge
antic                     0.2.5                h44d021b_2    conda-forge
arb                       2.23.0               h44d021b_0    conda-forge
bzip2                     1.0.8                hd590300_5    conda-forge
ca-certificates           2023.11.17           hbcca054_0    conda-forge
gf2x                      1.3.0                ha476b99_2    conda-forge
gmp                       6.3.0                h59595ed_0    conda-forge
ld_impl_linux-64          2.40                 h41732ed_0    conda-forge
libeantic                 1.3.0                h39cd0c9_1    conda-forge
libexpat                  2.5.0                hcb278e6_1    conda-forge
libffi                    3.4.2                h7f98852_5    conda-forge
libflint                  2.9.0           h2f819a4_ntl_100    conda-forge
libgcc-ng                 13.2.0               h807b86a_3    conda-forge
libgomp                   13.2.0               h807b86a_3    conda-forge
libnsl                    2.0.1                hd590300_0    conda-forge
libsqlite                 3.44.2               h2797004_0    conda-forge
libstdcxx-ng              13.2.0               h7e041cc_3    conda-forge
libuuid                   2.38.1               h0b41bf4_0    conda-forge
libxcrypt                 4.4.36               hd590300_1    conda-forge
libzlib                   1.2.13               hd590300_5    conda-forge
mpfr                      4.2.1                h9458935_0    conda-forge
ncurses                   6.4                  h59595ed_2    conda-forge
normaliz                  3.10.1               h5975b00_0    conda-forge
ntl                       11.4.3               hef3c4d3_1    conda-forge
openssl                   3.2.0                hd590300_1    conda-forge
pynormaliz                2.17            py312h8572e83_3    conda-forge
python                    3.12.1          hab00c5b_1_cpython    conda-forge
python_abi                3.12                    4_cp312    conda-forge
readline                  8.2                  h8228510_1    conda-forge
tk                        8.6.13          noxft_h4845f30_101    conda-forge
tzdata                    2023d                h0c530f3_0    conda-forge
xz                        5.2.6                h166bdaf_0    conda-forge

Environment info

active environment : pynormaliz
    active env location : /home/jule/proj/umamba/envs/pynormaliz
            shell level : 1
       user config file : /home/jule/.condarc
 populated config files : /home/jule/.condarc
          conda version : 23.11.0
    conda-build version : 3.28.2
         python version : 3.11.7.final.0
                 solver : libmamba (default)
       virtual packages : __archspec=1=icelake
                          __conda=23.11.0=0
                          __glibc=2.38=0
                          __linux=6.6.11=0
                          __unix=0=0
       base environment : /home/jule/proj/umamba  (writable)
      conda av data dir : /home/jule/proj/umamba/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
          package cache : /home/jule/proj/umamba/pkgs
                          /home/jule/.conda/pkgs
       envs directories : /home/jule/proj/umamba/envs
                          /home/jule/.conda/envs
               platform : linux-64
             user-agent : conda/23.11.0 requests/2.31.0 CPython/3.11.7 Linux/6.6.11-2-lts arch/rolling glibc/2.38 solver/libmamba conda-libmamba-solver/23.12.0 libmambapy/1.5.5
                UID:GID : 1000:1000
             netrc file : None
           offline mode : False
saraedum commented 7 months ago

I think we should just make the run exports of normaliz x.x or even x.x.x. I don't think the normaliz authors worry too much about ABI compatibility, e.g., there's a field added to a public class in a patch release here https://github.com/Normaliz/Normaliz/compare/3.10.0...v3.10.1#diff-20e42966758eba44ffc1b2bd8457b4831ef7338a71cde0ed00b7f8b8614f34a3R673. (Unless I miss how the source code is organized.)

saraedum commented 7 months ago

@isuruf, what do you think?

isuruf commented 7 months ago

Sure. Sounds good.

saraedum commented 7 months ago

I created https://github.com/conda-forge/normaliz-feedstock/pull/51 for this. Afterwards, we still need to patch the repodata. But let's fix the recipe first…