Skip to content

Conversation

@MargaretDuff
Copy link
Member

@MargaretDuff MargaretDuff commented Jan 29, 2026

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
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.

1 participant