emichael / dslabs

Distributed Systems Labs and Framework
https://ellismichael.com/dslabs/
1.23k stars 339 forks source link

Don't replace timers on update, highlight new timers #25

Closed emichael closed 2 years ago

emichael commented 2 years ago

The new viz tool currently replaces all timers on any state change. It also does not highlight newly added timers like it does with messages. Part of the issue is that timers are trickier to deal with since queues are semi-ordered and can have duplicates.

See: https://github.com/emichael/dslabs/blob/54f9fc1bb43bfc9c2f429e90ac8ec36ce59ea002/framework/tst/dslabs/framework/testing/newviz/SingleNodePanel.java#L175-L187

emichael commented 2 years ago

Fixed in ff055e2a5cb1285511d649f6dd1c4c9cfc02c118.