Skip to content

Festim as spack package#4

Merged
jorgensd merged 3 commits intomainfrom
dokken/festim
Feb 25, 2026
Merged

Festim as spack package#4
jorgensd merged 3 commits intomainfrom
dokken/festim

Conversation

@jorgensd
Copy link
Member

@jorgensd jorgensd commented Feb 22, 2026

@RemDelaporteMathurin @jhdark.
I've made a spack package for FESTIM.
"main" branch i.e. fenicsx branch fails with:

================================================================================================= short test summary info =================================================================================================
FAILED test/system_tests/test_io.py::test_writing_and_reading_of_species_function_using_checkpoints - RuntimeError: Meshes do no match.
FAILED test/system_tests/test_multi_mat_penalty.py::test_derived_quantities_multi_mat - KeyError: Mesh(blocked element (Basix element (P, triangle, 1, gll_warped, unset, False, float64, []), (2,)), 62)
FAILED test/test_h_transport_problem.py::test_surface_reaction_BC_discontinuous - KeyError: Mesh(blocked element (Basix element (P, interval, 1, gll_warped, unset, False, float64, []), (1,)), 126)
FAILED test/test_initial_condition.py::test_checkpointing_single_species - RuntimeError: Meshes do no match.
FAILED test/test_initial_condition.py::test_checkpointing_multiple_species - RuntimeError: Meshes do no match.
============================================================================ 5 failed, 649 passed, 1 skipped, 13 warnings in 172.14s (0:02:52)

while 2.0.0-beta.2post1 passes all tests.
Spack env file:

# This is a Spack Environment file.
#
# It describes a set of packages to be installed, along with
# configuration settings.
spack:
  # add package specs to the `specs` list
  specs:
  - py-festim@2.0
  - py-pytest
  - py-sympy
  - py-ipyparallel
  - py-pip
  view: true
  concretizer:
    unify: true
  repos:
    scientificcomputing: /home/dokken/Documents/spack_repos/spack_repo/scientificcomputing
    fenics:
      git: https://github.com/FEniCS/spack-fenics.git
  packages:
    gcc:
      externals:
      - spec: gcc@13.3.0 languages:='c,c++,fortran'
        prefix: /usr
        extra_attributes:
          compilers:
            c: /usr/bin/gcc
            cxx: /usr/bin/g++
            fortran: /usr/bin/gfortran

@RemDelaporteMathurin
Copy link

I don't understand why fenicsx would fail, it's extremely similar to 2.0.0-beta.2post

festim-dev/FESTIM@v2.0-beta.2post1...fenicsx

@jorgensd
Copy link
Member Author

I don't understand why fenicsx would fail, it's extremely similar to 2.0.0-beta.2post

festim-dev/FESTIM@v2.0-beta.2post1...fenicsx

I've pinned beta to work with 0.10 release, while the fenicsx branch aims for the main branch. The issue is probably the misuse of interpolating between grids in FESTIM.

@RemDelaporteMathurin
Copy link

Ah yes, we don't "aim" for the main branch of dolfinx per se, but we have it in the CI to have a headstart on the upgrade work when a new dolfinx version is released.

@jorgensd jorgensd merged commit 78a6333 into main Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants