search

slac-lcls / ami

Other
6 stars 3 forks source link

Fix sending multiple checkpoints. #76

Closed syamajala closed 1 month ago

syamajala commented 2 months ago

This should fix sending multiple checkpoints each time the graph is submitted.

It also cleans up some stuff and removes passing the node to the trace editor for plots.

I am still investigating the roi.

syamajala commented 2 months ago

I found a case where the state of the trace editor is not preserved properly. This fix is not correct.

syamajala commented 2 months ago

@vespos can you try this pull request and see if it fixes the sending multiple checkpoint problem?

I realized that since I use a tiling window manager, it was causing extra checkpoints to be emitted, switching to a floating layout it seems like maybe this is the correct fix and its working properly?

syamajala commented 2 months ago

This branch does not seem to fix the problem of checkpoints accumulating when the graph changes.

syamajala commented 1 month ago

This branch has been merged.