Firedrake meeting 2025 04 02 - firedrakeproject/firedrake GitHub Wiki
Date and time 2025-04-02 1600 UTC
Action Items
- Pick Chair and Minuter (IM 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)
Agenda
Present:
Apologies:
Release
https://github.com/orgs/firedrakeproject/projects/9
Firedrake #4177. Subfunctions adjoint evaluation.
JHCDescription in PR. More fiddly FloatingType
details again.
Minutes
JHC: simple fix but involved lots of work to figure it out so there's a long description in the PR.
CW: Add a comment in the code with similar level of detail as the PR description.
CW: self.idx -> self.subfunction_index in FunctionMergeBlock
.
CW petsc4py suggestion
#4175 (information in PR summary)
Be warned: be prepared to think hard pip thoughts
Minutes
CW: Proposal: a shim package petsctools
to install petsc4py directly from PETSc (as opposed to using pip).
DH: There are problems with a separate PETSc install.
DH: We can pip install petsc4py by specifying the system PETSc dir. For Ubuntu the plan is to install a PPA PETSc (much harder for Mac).
DH: As long as the user can override the specific version, we could by default dynamically figure out the PETSc version.
DH: We need to consider this further. The user should might not get the versions right.
CW: We consider more flexible minimal PETSc requirements/versions.
DH: Firedrake release branch should be as new user friendly as possible.
CW: I'll think some more.
PB: UFL performance regression
https://github.com/FEniCS/ufl/pull/367/files
PB: I still need to get the caching interface.
DH: use object id to memoize visited nodes in the tree.
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.
-
JHC: Firedrake #4178. Only pass solver parameters to adjoint solve block once. Merged.
-
KS: https://github.com/FEniCS/ufl/pull/368. Cleanup BaseFormOperator arguments. Merged.
-
CW: https://github.com/firedrakeproject/firedrake/pull/4189 We check links in the docs, sometimes we get failures due to the other sites changing, other times sites are temporarily down. This PR deploys the website even if the link check fails.
-
PB: Restrict PC Not discussed
-
PB: FIAT mimetic spectral Merged
-
DH: Adjoint documentation Merged
-
DH: Fix streamline plots Reviewed, merge when it passes CI
1600 UTC 2025-04-23