Firedrake meeting 2025 03 19 - firedrakeproject/firedrake GitHub Wiki
Date and time 2025-03-19 1600 UTC
Action Items
- Pick Chair and Minuter (JHC to pick)
- ALL: (ongoing) triage the open issues and confirm if they are indeed still open (and perhaps provide labels)
- ALL: do things with SV's branches
- DH: Email to Andreas to have 2 (+ others!!!) loopy PRs merged TODO: FIND OUT WHICH PRS THESE ARE
- DH: Get Firedrake a docker open source account (link here)
- DH: Talk to GregVernon about PR#2116.
- JB: Enable merge queues (minutes)
- LC: Try to merge RNH' PR: Movable VOM
- PB: Profile and speed up some tests (minutes, minutes)
- CW: More testing configurations (minutes)
Minutes
Present: DH, CW, RK, KS, PB, IM, JHC, DD, AO, LC
Apologies:
CW: Root in Docker
https://github.com/firedrakeproject/firedrake/pull/4104
Seems to work fairly well. I have tested with a number of downstream packages.
- Slepc moved to
/opt/slepc
- Approved
- Make announcement after merge
CW: Remove part of CI
https://github.com/firedrakeproject/firedrake/pull/4086
- Merged
JHC: Various adjoint fixes
#4126 Ensure Function.subfunctions
is always taped. Currently, subfunctions are not taped if they are first accessed before annotation is enabled. This PR corrects that and adds a test.
- Approved
#4128 Variational solver should either complain if Merged.bounds
are passed to solve
, or remove them from the kwargs
if they are None
.
#4130 Only pass solver_parameters
to the adjoint solver once, and make sure it's the one processed by OptionsManager
.
- Merged
Merge PRs
Note that PRs put in this section should either be trivial or already have been reviewed. Discussion-worthy PRs should be separate agenda items.
__init__.py
to be robust to not detecting BLAS #4083
CW: Cleanup - Raises warning for failing to set blas
- Merged
FIAT #134
IM: FIAT general hypercube- Merged
#4115
AO: Add spyro smoke tests- Takes 5 mins max (JHC says 1m20s)
- Approved
#4012
PB: Make LinearSolver a subclass of LinearVariationalSolver- approved previously, but was failing tests
- Approved
#4056
PB: Assemble: use tensor kwarg in FormSum maxpy- Discussed previously
- matrix add now returns
AssembledMatrix
- Merged
#3973
PB: Boundary Quadrature elements- Need to add docstring to
MappedPointSet
- Makes
gem.Product
n-ary (syntax sugar) - Don't abbreviate variable names
- Looks good overall
#4131
CW: Runners have 8 threads, not 12- Approved
- #4086 should fix the failure
Release discussion
- CW: Going to try move off petsc fork to petsc main for release
- DH: release after easter. Deprecated APIs will be dropped e.g.
interpolate
,.at()
Date of next meeting
1600 UTC 2025-03-26??? (TBD, many core developers will be away) (Tentative 14:00)