Skip to content

Dham/abstract reduced functional#3941

Merged
JHopeCollins merged 92 commits intomainfrom
dham/abstract_reduced_functional
Aug 13, 2025
Merged

Dham/abstract reduced functional#3941
JHopeCollins merged 92 commits intomainfrom
dham/abstract_reduced_functional

Conversation

@dham
Copy link
Member

@dham dham commented Dec 18, 2024

Relies on: dolfin-adjoint/pyadjoint#156

The Firedrake changes are mostly internal, or only exposed if a user specifically requests them.

  • New RieszMap object for storing a persistent solver.
  • Refactor of EnsembleReducedFunctional to inherit from AbstractReducedFunctional
  • Updates to the Function and Cofunction adjoint mixins.

@github-actions
Copy link

github-actions bot commented Dec 18, 2024

TestsPassed ✅Skipped ⏭️Failed ❌
Firedrake complex7833 ran6369 passed1464 skipped0 failed

@github-actions
Copy link

github-actions bot commented Dec 18, 2024

TestsPassed ✅Skipped ⏭️Failed ❌
Firedrake real8003 ran7270 passed711 skipped22 failed

JHopeCollins
JHopeCollins previously approved these changes Aug 12, 2025
Copy link
Member

@JHopeCollins JHopeCollins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ready to go. The only CI failure is an unrelated linkcheck

JHopeCollins
JHopeCollins previously approved these changes Aug 13, 2025
Copy link
Member

@JHopeCollins JHopeCollins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ready to go. The only CI failure is an unrelated linkcheck

@JHopeCollins JHopeCollins merged commit 79900ff into main Aug 13, 2025
2 of 4 checks passed
@JHopeCollins JHopeCollins deleted the dham/abstract_reduced_functional branch August 13, 2025 14:41
angus-g pushed a commit to angus-g/firedrake that referenced this pull request Aug 19, 2025
Co-authored-by: Daiane Iglesia Dolci <63597005+Ig-dolci@users.noreply.github.com>
Co-authored-by: Josh Hope-Collins <joshua.hope-collins13@imperial.ac.uk>
Co-authored-by: Pablo Brubeck <brubeck@protonmail.com>
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.

4 participants