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-2476] First graph run does not show execution time in TuneUp + #47

Conversation

ivaylo-matov
Copy link
Contributor

Small PR that aims to address https://jira.autodesk.com/browse/DYN-2476 , as well as (partially) https://jira.autodesk.com/browse/DYN-4340 🤞

DYN-2476_1

DYN-0000_1

  • TuneUp tracks changes in workspaces but only subscribes to these changes when it is enabled.
  • When enabled, TuneUp forces the workspace to use the Manual run type to improve performance.
  • Removed unnecessary updates to the UI to enhance responsiveness.
  • Node execution time is now measured using Stopwatch for better accuracy.
  • DataGrid headers retain their sorting order when the graph is re-executed. The sorting order reverts to the default when TuneUp is disabled, preparing for further improvements in https://jira.autodesk.com/browse/DYN-2475

TODO:
- Remove mid-run update
-Keep Execution time on current tun on top or bottom when sorting by name
@QilongTang
Copy link
Contributor

Thanks @ivaylo-matov Would you resolve the merge conficts?

@ivaylo-matov
Copy link
Contributor Author

All done @QilongTang 👍
There'll probably be more conflicts on the other PRs. I'll check them out later

@reddyashish reddyashish merged commit 0d8e8d8 into DynamoDS:master Jul 17, 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.

3 participants