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

[DYN-2654] groups of nodes in graph should be displayed as group in tuneup list view #51

Conversation

ivaylo-matov
Copy link
Contributor

Aims to address https://jira.autodesk.com/browse/DYN-2654 🤞

DYN-2654_PR

Displays Groups in TuneUp:

  • Each grouped node is indented and positioned under the group header, regardless of the sorting order.
  • Sorting can be based on total group execution time, group execution order, or group name.
  • When sorting by execution order, if a group is removed, the execution order of the previously grouped nodes will not display, but the nodes will remain in the same location.
  • When nodes are grouped, TuneUp will display the group and place the nodes under the header.
  • The group will initially have its default name. If the name is changed, TuneUp will update it on the next graph run.
  • If the color is changed, TuneUp will update it the next time the graph is loaded.

Hope that's okay.

Copy link
Contributor

@reddyashish reddyashish left a comment

Choose a reason for hiding this comment

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

LGTM

@reddyashish reddyashish merged commit f578164 into DynamoDS:master Jul 26, 2024
10 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.

2 participants