Skip to content

Add a draw method to produce graphviz images - #324

Merged
liamhuber merged 12 commits into
mainfrom
draw
Sep 8, 2026
Merged

liamhuber merged 12 commits into
mainfrom
draw

Conversation

@liamhuber

@liamhuber liamhuber commented Sep 8, 2026 •

Copy link
Copy Markdown
Member

For both prospective recipes and retrospective data.

E.g.,
quadratic

Closes #318

liamhuber and others added 11 commits September 7, 2026 13:59
Just centralizes some string management, and nicely gives the path delimiter a single source of truth

Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
Signed-off-by: Liam Huber <liamhuber@greyhavensolutions.com>
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Binder 👈 Launch a binder notebook on branch pyiron/flowrep/draw

@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.95%. Comparing base (47f0681) to head (6be8147).

Additional details and impacted files
@@           Coverage Diff            @@
##             main     #324    +/-   ##
========================================
  Coverage   99.94%   99.95%            
========================================
  Files          44       52     +8     
  Lines        3590     4016   +426     
========================================
+ Hits         3588     4014   +426     
  Misses          2        2            

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@liamhuber

Copy link
Copy Markdown
Member Author

@samwaseda, FYI this exists now! We can polish up the appearance going forward, but this is cleanly isolated and an optional dependency, so I'm going to carry straight on to the merge.

@liamhuber
liamhuber merged commit b3fdac9 into main Sep 8, 2026
22 checks passed
@liamhuber
liamhuber deleted the draw branch September 8, 2026 01:01
@liamhuber

Copy link
Copy Markdown
Member Author

For posterity,

import flowrep as fr

@fr.workflow
def do_while(x0, limit, step):
    x = x0
    while fr.std.lt(x, limit):
        x = fr.std.add(x, step)
    return x

do_while.flowrep_recipe.draw()
while

Which is, IMO, solidly in "good enough"-territory

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.

Add a drawing tool

2 participants