Skip to content

Conversation

sanjay78632
Copy link
Contributor

What changes are being made and why?

This PR improves the readability of the Dependencies view by tuning the Cytoscape COSE layout parameters:

  • Increased nodeRepulsion to push nodes further apart
  • Increased idealEdgeLength for longer links
  • Increased componentSpacing and padding for clearer clustering
  • Increased numIter for more stable layout convergence

These changes address the issue where large graphs were too dense and difficult to explore.
Closes #11583.


How the changes have been QAed?

Tested locally using the sample parent/child/grandchild flows from PR #10240.
With the updated layout settings, the dependency graph is easier to read and nodes are spread out more clearly.

Example flows used:

id: parent
namespace: graph.qa
tasks:
  - id: subflow
    type: io.kestra.plugin.core.flow.Subflow
    namespace: graph.qa
    flowId: child

id: child
namespace: graph.qa
tasks:
  - id: subflow
    type: io.kestra.plugin.core.flow.Subflow
    namespace: demo
    flowId: grandchild

id: grandchild
namespace: demo
tasks:
  - id: hello
    type: io.kestra.plugin.core.log.Log
    message: "Hello World! 🚀"

@MilosPaunovic MilosPaunovic self-requested a review October 9, 2025 11:04
@sanjay78632
Copy link
Contributor Author

Hi 👋 @MilosPaunovic this PR is ready from my side. Please let me know if you’d like me to tweak the values further.

This is only my second PR in Kestra and my first time working on the dependencies graph layout, so I’m still learning the ropes 😅. Really happy to contribute and open to any feedback or adjustments you’d like me to make

Copy link
Member

@MilosPaunovic MilosPaunovic left a comment

Choose a reason for hiding this comment

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

Hi @sanjay78632,

This maybe helped out a little, but not enough yet.

image

You need to test it with at least 500 dependencies, that's why I suggested to use mock data here #11583 (comment).

@MilosPaunovic MilosPaunovic moved this from To review to On hold in Pull Requests Oct 10, 2025
@MilosPaunovic
Copy link
Member

Can you provide screenshot of how it looks now @sanjay78632?

@MilosPaunovic MilosPaunovic moved this from On hold to To review in Pull Requests Oct 12, 2025
@MilosPaunovic MilosPaunovic self-requested a review October 12, 2025 06:27
@MilosPaunovic MilosPaunovic moved this from To review to On hold in Pull Requests Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: On hold

Development

Successfully merging this pull request may close these issues.

Add repel and link forces to dependencies view

2 participants