Skip to content

Conversation

@TimMonko
Copy link
Member

@TimMonko TimMonko commented Dec 22, 2025

This is a re-implementation of https://github.com/haesleinhuepf/napari-workflow-inspector

This version maintains the same matplotlib style task graph inspector. It has the addition of uploading a workflow.yaml to inspect, which previous versions didn't. It's npe2 compatible and modern napari ready. It has not been validated for its work yet with live workflow production (like with napari-assistant), but that code path is hopefully in tact.

Because this adds no dependencies, I thought it fine to just live in this repo, but it could be refactored out in the future.

This was generated mostly with the help of copilot, and perhaps I will regret adding this because I have little interest in actively maintaining qt code, instead preferring magicgui widgets and such. But, I do think this is most helpful for collaborators, and is a sanity check that my new workflow format is set up correctly.

The plot won't generate without matplotlib, but other parts of the inspector work, so I added it as an optional to keep dependencies slimmer for now. Most environments seem to end up with matplotlib eventually so I'm not too worried.

@codecov
Copy link

codecov bot commented Dec 22, 2025

Codecov Report

❌ Patch coverage is 81.36364% with 82 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/ndev_workflows/widgets/_workflow_inspector.py 80.93% 82 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@TimMonko TimMonko merged commit 169bc71 into ndev-kit:main Dec 26, 2025
11 of 12 checks passed
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.

1 participant