DynamoDS / TuneUp

A profiler for Dynamo graphs
Apache License 2.0
27 stars 15 forks source link

[DYN-4517] TuneUp will fail to register if not closed between graph runs #43

Closed ivaylo-matov closed 3 months ago

ivaylo-matov commented 3 months ago

PR aims to address https://jira.autodesk.com/browse/DYN-4517. It also addresses 1. of https://autodesk.slack.com/archives/CPJ5UQW0Z/p1719284511762389 : TuneUp to not impact performance when not being actively used.

DYN-4517_Fix

FYI @QilongTang @Ashish Ganta @Amoursol

reddyashish commented 3 months ago

@ivaylo-matov there are still 2 conflicts.

ivaylo-matov commented 3 months ago

@reddyashish , sorry. Tried to resolve them before a couple of times, but wasn't sure if I should Commit Merge.

reddyashish commented 3 months ago

LGTM. Thank you @ivaylo-matov.

reddyashish commented 3 months ago

@QilongTang Will open a new PR to publish the tuneup package.