Skip to content

feat: visualize runs... <- (add colouring to edges) #3383

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: 3103-visualize-runs-on-the-canvas
Choose a base branch
from

Conversation

doc-han
Copy link
Contributor

@doc-han doc-han commented Jul 9, 2025

Description

This is a POC to add some colouring to edges when visualizing runs on the canvas.

Screenshot 2025-07-09 at 6 20 27 PM

Closes #__

Validation steps

  1. (How can a reviewer validate your work?)

Additional notes for the reviewer

  1. (Is there anything else the reviewer should know or look out for?)

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our Responsible AI Policy

Pre-submission checklist

  • I have performed a self-review of my code.
  • I have implemented and tested all related authorization policies. (e.g., :owner, :admin, :editor, :viewer)
  • I have updated the changelog.
  • I have ticked a box in "AI usage" in this PR

@github-project-automation github-project-automation bot moved this to New Issues in v2 Jul 9, 2025
@doc-han doc-han marked this pull request as draft July 9, 2025 18:21
@doc-han doc-han changed the title feat: add coloring to edges feat: visualize runs... <- (add colouring to edges) Jul 9, 2025
@josephjclark
Copy link
Collaborator

josephjclark commented Jul 10, 2025

Some problems with this approach:

  • We don't really know whether an edge ran or not. In the screenshot above, that red edge didn't actually trigger
  • That information doesn't exist at all
  • What colour is an edge if it's a condition, rather than on fail?

We can't really do this until we get more information OpenFn/kit#956

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New Issues
Development

Successfully merging this pull request may close these issues.

2 participants