Skip to content

feat: add live volumewise registration plots - #352

Open
sdiebolt wants to merge 6 commits into
mainfrom
feat/live-volumewise-progress
Open

feat: add live volumewise registration plots#352
sdiebolt wants to merge 6 commits into
mainfrom
feat/live-volumewise-progress

Conversation

@sdiebolt

@sdiebolt sdiebolt commented Jul 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Summary

  • add live matplotlib motion diagnostics for register_volumewise
  • rename registration progress protocols/classes for volume vs volumewise clarity
  • preserve time units in motion diagnostics plots

Closes #349.

@sdiebolt
sdiebolt requested a review from FelipeCybis July 31, 2026 22:48
@sdiebolt sdiebolt self-assigned this Jul 31, 2026
@sdiebolt sdiebolt added the enhancement New feature or request label Jul 31, 2026
@sdiebolt sdiebolt changed the title Add live volumewise registration plots feat: add live volumewise registration plots Jul 31, 2026
@sdiebolt
sdiebolt force-pushed the feat/live-volumewise-progress branch 2 times, most recently from 9718085 to 77e0b37 Compare July 31, 2026 22:51
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

📖 Doc preview: https://confusius.tools/pr-preview/pr-352/

@sdiebolt
sdiebolt force-pushed the feat/live-volumewise-progress branch 2 times, most recently from bf57e29 to 941a528 Compare July 31, 2026 23:14
@codecov

codecov Bot commented Jul 31, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sdiebolt
sdiebolt force-pushed the feat/live-volumewise-progress branch from 941a528 to 021c583 Compare July 31, 2026 23:57
@sdiebolt
sdiebolt requested a review from Copilot July 31, 2026 23:57
Copilot AI reviewed Jul 31, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sdiebolt
sdiebolt marked this pull request as ready for review July 31, 2026 23:58
@sdiebolt

sdiebolt commented Jul 31, 2026

Copy link
Copy Markdown
Member Author

@FelipeCybis Ready to review!

@sdiebolt
sdiebolt force-pushed the feat/live-volumewise-progress branch from 021c583 to 6b0f931 Compare August 1, 2026 00:01
@sdiebolt
sdiebolt force-pushed the feat/live-volumewise-progress branch from ed7976a to 5d6aafd Compare August 3, 2026 08:19

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@FelipeCybis FelipeCybis Awaiting requested review from FelipeCybis

At least 1 approving review is required to merge this pull request.

Assignees

@sdiebolt sdiebolt

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add a live progress plotter to register_volumewise

3 participants

@sdiebolt @FelipeCybis