Skip to content
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

feat(ui) : Change edge type from smoothstep to bezier #5350 #5606

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

sanketmagar2001
Copy link

@sanketmagar2001 sanketmagar2001 commented Oct 23, 2024

changing edge type from smoothstep to bezier.

What changes are being made and why?

Changing edge type from 'smoothstep' to 'bezier' for more readable in complex dependency graph.

How the changes have been QAed?


Setup Instructions

Closes #5350.

@sanketmagar2001
Copy link
Author

PR in ui-libs :kestro-io/ui-libs changes

@Skraye
Copy link
Member

Skraye commented Oct 25, 2024

Hello @sanketmagar2001, unfortunately, the issue is still there :
image

@Skraye
Copy link
Member

Skraye commented Nov 12, 2024

@sanketmagar2001 Hello, any update on this?

@sanketmagar2001
Copy link
Author

Hi @Skraye, apologies for the delay 🙂. I was celebrating the festive season in our country, so I couldn't work on this during that time.

@sanketmagar2001
Copy link
Author

We can also apply the approach 2 which I have mention : Calculate Edge Curvature and Offset , this may work as we can calculate the offset from the node . I will try this out , and let you know . will also update the PR once done.

@sanketmagar2001
Copy link
Author

for now let's try with hard coded values , if this does not work . we can calculate the offset and curvature from source to target node.

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

Successfully merging this pull request may close these issues.

Better display of topology (especially for the Dependencies)
3 participants