Skip to content

Add a block diagonal operator#2279

Draft
MargaretDuff wants to merge 3 commits into
TomographicImaging:masterfrom
MargaretDuff:digonalidentityoperator
Draft

Add a block diagonal operator#2279
MargaretDuff wants to merge 3 commits into
TomographicImaging:masterfrom
MargaretDuff:digonalidentityoperator

Conversation

@MargaretDuff

@MargaretDuff MargaretDuff commented Jan 29, 2026

Copy link
Copy Markdown
Member

Changes

The diagonal operator checks if a block data container is passed to it and then either goes to the old diagonal operator or a new one, which deals with the block operations (without ZeroOperators).

Written with the help of @bryceshirley

To do for merging into CIL

  • Add unit tests including when you have blocks of blocks
  • Check the documentation (it was written in a rush)

Testing you performed

Please add any demo scripts to https://github.com/TomographicImaging/CIL-Demos/tree/main/misc

Related issues/links

Reopens a very old issue #1568

Checklist

  • I have performed a self-review of my code
  • I have added docstrings in line with the guidance in the developer guide
  • I have updated the relevant documentation
  • I have implemented unit tests that cover any new or modified functionality
  • CHANGELOG.md has been updated with any functionality change
  • Request review from all relevant developers

--->

@MargaretDuff MargaretDuff self-assigned this Jan 29, 2026
@casperdcl casperdcl moved this to In Progress in UM 2026 Jan 29, 2026
@lauramurgatroyd lauramurgatroyd marked this pull request as draft June 5, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants