Skip to content

Conversation

@jcortial-safran
Copy link
Contributor

No description provided.

Up to 9.15.0, medcoupling offically builds with numpy 1.y.z, so it seems
reasonable to require numpy's major version to be 1.
Furthermore, 9.13.0 actually fails to build when ^py-numpy@2.

medcoupling 9.11.0 and 9.12.0 officially build with scipy 0.19.1, and
9.13.0 up to 9.15.0 with scipy 1.6.2.
In practice, 9.13.0 acutally fails to build when ^[email protected] but 9.11.0
and 9.13.0 build successfully when ^[email protected].
Minimum required CMake version is 2.8.11, incompatible with CMake 4
Make the official python version the lower bound
@jcortial-safran
Copy link
Contributor Author

@spackbot help

@spackbot-app
Copy link

spackbot-app bot commented Oct 25, 2025

You can interact with me in many ways!

  • @spackbot hello: say hello and get a friendly response back!
  • @spackbot help or @spackbot commands: see this message
  • @spackbot run pipeline or @spackbot re-run pipeline: to request a new run of the GitLab CI pipeline
  • @spackbot rebuild everything: to run a pipeline rebuilding all specs from source.
  • @spackbot fix style if you have write and would like me to run spack style --fix for you.
  • @spackbot maintainers or @spackbot request review: to look for and assign reviewers for the pull request.

I'll also help to label your pull request and assign reviewers!
If you need help or see there might be an issue with me, open an issue here

@jcortial-safran
Copy link
Contributor Author

@spackbot fix style

@spackbot-app
Copy link

spackbot-app bot commented Oct 25, 2025

Let me see if I can fix that for you!

@spackbot-app
Copy link

spackbot-app bot commented Oct 25, 2025

I was able to run spack style --fix for you!

spack style --fix
repos/spack_repo/builtin/packages/py_cffi/package.py
repos/spack_repo/builtin/packages/py_frozenlist/package.py
repos/spack_repo/builtin/packages/py_multidict/package.py
repos/spack_repo/builtin/packages/py_parso/package.py
repos/spack_repo/builtin/packages/py_pygtrie/package.py
repos/spack_repo/builtin/packages/salome_configuration/package.py
repos/spack_repo/builtin/packages/salome_medcoupling/package.py
�[1;34m==> �[0mrunning flake8
repos/spack_repo/builtin/packages/salome_medcoupling/package.py:123: [E231] missing whitespace after ','
�[1;34m==> �[0mrunning isort
�[1;34m==> �[0mrunning black
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I've updated the branch with style fixes.

alalazo
alalazo previously approved these changes Oct 28, 2025
@alalazo alalazo self-assigned this Oct 28, 2025
@alalazo
Copy link
Member

alalazo commented Oct 28, 2025

Giving a few days to @franciskloss to review

@jcortial-safran
Copy link
Contributor Author

@spackbot fix style

@spackbot-app
Copy link

spackbot-app bot commented Oct 28, 2025

Let me see if I can fix that for you!

@spackbot-app
Copy link

spackbot-app bot commented Oct 28, 2025

I was able to run spack style --fix for you!

spack style --fix
repos/spack_repo/builtin/packages/amdsmi/package.py
repos/spack_repo/builtin/packages/cdi/package.py
repos/spack_repo/builtin/packages/cdo/package.py
repos/spack_repo/builtin/packages/comgr/package.py
repos/spack_repo/builtin/packages/composable_kernel/package.py
repos/spack_repo/builtin/packages/cudss/package.py
repos/spack_repo/builtin/packages/cxxopts/package.py
repos/spack_repo/builtin/packages/ectrans/package.py
repos/spack_repo/builtin/packages/fast_float/package.py
repos/spack_repo/builtin/packages/fenics_basix/package.py
repos/spack_repo/builtin/packages/freetype/package.py
repos/spack_repo/builtin/packages/gaudi/package.py
repos/spack_repo/builtin/packages/geopm_service/package.py
repos/spack_repo/builtin/packages/geos/package.py
repos/spack_repo/builtin/packages/ghostscript/package.py
repos/spack_repo/builtin/packages/google_cloud_cli/package.py
repos/spack_repo/builtin/packages/graphite2/package.py
repos/spack_repo/builtin/packages/grpc/package.py
repos/spack_repo/builtin/packages/harfbuzz/package.py
repos/spack_repo/builtin/packages/hdf5/package.py
repos/spack_repo/builtin/packages/hip/package.py
repos/spack_repo/builtin/packages/hip_tensor/package.py
repos/spack_repo/builtin/packages/hip_tests/package.py
repos/spack_repo/builtin/packages/hipblas/package.py
repos/spack_repo/builtin/packages/hipblas_common/package.py
repos/spack_repo/builtin/packages/hipblaslt/package.py
repos/spack_repo/builtin/packages/hipcc/package.py
repos/spack_repo/builtin/packages/hipcub/package.py
repos/spack_repo/builtin/packages/hipfft/package.py
repos/spack_repo/builtin/packages/hipfort/package.py
repos/spack_repo/builtin/packages/hipify_clang/package.py
repos/spack_repo/builtin/packages/hiprand/package.py
repos/spack_repo/builtin/packages/hipsolver/package.py
repos/spack_repo/builtin/packages/hipsparse/package.py
repos/spack_repo/builtin/packages/hipsparselt/package.py
repos/spack_repo/builtin/packages/hsa_amd_aqlprofile/package.py
repos/spack_repo/builtin/packages/hsa_rocr_dev/package.py
repos/spack_repo/builtin/packages/hypre/package.py
repos/spack_repo/builtin/packages/intel_oneapi_advisor/package.py
repos/spack_repo/builtin/packages/intel_oneapi_ccl/package.py
repos/spack_repo/builtin/packages/intel_oneapi_compilers/package.py
repos/spack_repo/builtin/packages/intel_oneapi_dnn/package.py
repos/spack_repo/builtin/packages/intel_oneapi_dpct/package.py
repos/spack_repo/builtin/packages/intel_oneapi_dpl/package.py
repos/spack_repo/builtin/packages/intel_oneapi_ipp/package.py
repos/spack_repo/builtin/packages/intel_oneapi_ippcp/package.py
repos/spack_repo/builtin/packages/intel_oneapi_mkl/package.py
repos/spack_repo/builtin/packages/intel_oneapi_mpi/package.py
repos/spack_repo/builtin/packages/intel_oneapi_tbb/package.py
repos/spack_repo/builtin/packages/intel_oneapi_vtune/package.py
repos/spack_repo/builtin/packages/intel_tbb/package.py
repos/spack_repo/builtin/packages/iozone/package.py
repos/spack_repo/builtin/packages/libcint/package.py
repos/spack_repo/builtin/packages/libtirpc/package.py
repos/spack_repo/builtin/packages/llvm/package.py
repos/spack_repo/builtin/packages/llvm_amdgpu/package.py
repos/spack_repo/builtin/packages/mfem/package.py
repos/spack_repo/builtin/packages/migraphx/package.py
repos/spack_repo/builtin/packages/miopen_hip/package.py
repos/spack_repo/builtin/packages/mivisionx/package.py
repos/spack_repo/builtin/packages/octopus/package.py
repos/spack_repo/builtin/packages/openmpi/package.py
repos/spack_repo/builtin/packages/openssl/package.py
repos/spack_repo/builtin/packages/palace/package.py
repos/spack_repo/builtin/packages/pkg_config/package.py
repos/spack_repo/builtin/packages/podio/package.py
repos/spack_repo/builtin/packages/py_cffi/package.py
repos/spack_repo/builtin/packages/py_charset_normalizer/package.py
repos/spack_repo/builtin/packages/py_frozenlist/package.py
repos/spack_repo/builtin/packages/py_jax/package.py
repos/spack_repo/builtin/packages/py_jaxlib/package.py
repos/spack_repo/builtin/packages/py_jupyterlab/package.py
repos/spack_repo/builtin/packages/py_keras/package.py
repos/spack_repo/builtin/packages/py_mne/package.py
repos/spack_repo/builtin/packages/py_mpi4py/package.py
repos/spack_repo/builtin/packages/py_multidict/package.py
repos/spack_repo/builtin/packages/py_parse/package.py
repos/spack_repo/builtin/packages/py_parso/package.py
repos/spack_repo/builtin/packages/py_pycubexr/package.py
repos/spack_repo/builtin/packages/py_pygtrie/package.py
repos/spack_repo/builtin/packages/py_soupsieve/package.py
repos/spack_repo/builtin/packages/py_tinycss2/package.py
repos/spack_repo/builtin/packages/py_tomlkit/package.py
repos/spack_repo/builtin/packages/py_torchgeo/package.py
repos/spack_repo/builtin/packages/py_urllib3/package.py
repos/spack_repo/builtin/packages/py_uv/package.py
repos/spack_repo/builtin/packages/py_vcrpy/package.py
repos/spack_repo/builtin/packages/py_virtualenv/package.py
repos/spack_repo/builtin/packages/py_wcwidth/package.py
repos/spack_repo/builtin/packages/py_wesanderson/package.py
repos/spack_repo/builtin/packages/py_wxpython/package.py
repos/spack_repo/builtin/packages/py_yarl/package.py
repos/spack_repo/builtin/packages/r_nloptr/package.py
repos/spack_repo/builtin/packages/r_rbibutils/package.py
repos/spack_repo/builtin/packages/rccl/package.py
repos/spack_repo/builtin/packages/rdc/package.py
repos/spack_repo/builtin/packages/rdma_core/package.py
repos/spack_repo/builtin/packages/rocal/package.py
repos/spack_repo/builtin/packages/rocalution/package.py
repos/spack_repo/builtin/packages/rocblas/package.py
repos/spack_repo/builtin/packages/rocdecode/package.py
repos/spack_repo/builtin/packages/rocfft/package.py
repos/spack_repo/builtin/packages/rocjpeg/package.py
repos/spack_repo/builtin/packages/rocm_bandwidth_test/package.py
repos/spack_repo/builtin/packages/rocm_cmake/package.py
repos/spack_repo/builtin/packages/rocm_core/package.py
repos/spack_repo/builtin/packages/rocm_dbgapi/package.py
repos/spack_repo/builtin/packages/rocm_debug_agent/package.py
repos/spack_repo/builtin/packages/rocm_device_libs/package.py
repos/spack_repo/builtin/packages/rocm_examples/package.py
repos/spack_repo/builtin/packages/rocm_gdb/package.py
repos/spack_repo/builtin/packages/rocm_opencl/package.py
repos/spack_repo/builtin/packages/rocm_openmp_extras/package.py
repos/spack_repo/builtin/packages/rocm_smi_lib/package.py
repos/spack_repo/builtin/packages/rocm_tensile/package.py
repos/spack_repo/builtin/packages/rocm_validation_suite/package.py
repos/spack_repo/builtin/packages/rocminfo/package.py
repos/spack_repo/builtin/packages/rocmlir/package.py
repos/spack_repo/builtin/packages/rocprim/package.py
repos/spack_repo/builtin/packages/rocprofiler_compute/package.py
repos/spack_repo/builtin/packages/rocprofiler_dev/package.py
repos/spack_repo/builtin/packages/rocprofiler_register/package.py
repos/spack_repo/builtin/packages/rocprofiler_sdk/package.py
repos/spack_repo/builtin/packages/rocprofiler_systems/package.py
repos/spack_repo/builtin/packages/rocpydecode/package.py
repos/spack_repo/builtin/packages/rocrand/package.py
repos/spack_repo/builtin/packages/rocshmem/package.py
repos/spack_repo/builtin/packages/rocsolver/package.py
repos/spack_repo/builtin/packages/rocsparse/package.py
repos/spack_repo/builtin/packages/rocthrust/package.py
repos/spack_repo/builtin/packages/roctracer_dev/package.py
repos/spack_repo/builtin/packages/roctracer_dev_api/package.py
repos/spack_repo/builtin/packages/rocwmma/package.py
repos/spack_repo/builtin/packages/rpp/package.py
repos/spack_repo/builtin/packages/salome_configuration/package.py
repos/spack_repo/builtin/packages/salome_medcoupling/package.py
repos/spack_repo/builtin/packages/scitokens_cpp/package.py
repos/spack_repo/builtin/packages/sirius/package.py
repos/spack_repo/builtin/packages/starpu/package.py
repos/spack_repo/builtin/packages/theia_ide/package.py
repos/spack_repo/builtin/packages/transferbench/package.py
repos/spack_repo/builtin/packages/wxwidgets/package.py
repos/spack_repo/builtin/packages/yosys/package.py
repos/spack_repo/builtin/packages/zig/package.py
�[1;34m==> �[0mrunning flake8
repos/spack_repo/builtin/packages/salome_medcoupling/package.py:51: [E303] too many blank lines (2)
�[1;34m==> �[0mrunning isort
�[1;34m==> �[0mrunning black
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I've updated the branch with style fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants