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-7577] Removed groups with only notes inside #82

Conversation

ivaylo-matov
Copy link
Contributor

This PR aims to address DYN-7577.

Now, groups that only contain notes (and no nodes) won’t show up in TuneUp in these situations:

  • When opening a new graph that has "empty" groups
  • When enabling TuneUp on a graph with "empty" groups
  • When all nodes are removed from a group, leaving only notes behind

I've also added methods to create profiled groups to simplify the code and reduce some repetition.

DYN-7577-fix

@reddyashish
@QilongTang

@dnenov

@reddyashish reddyashish merged commit 460c280 into DynamoDS:master Nov 8, 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