Turns out that refactoring from relaying BrokerdFill from the openOrder sub wasn't tested all too well 😂.
Fixes a 2nd name error for the .broker_time field (which isn't provided in the incremental update msg so this just sets it as the brokerd time (from time.time()) for now; comments added about how we might approach getting the ownTrades (the original value we were passing) in the response message eventually.
Also included is a further fix to the paper engine to just avoid trying to pop simulated order book entries on modifies since it's not really necessary and further it was the source of a linger race-crash with the actual paper clear case (popped after already popped by clear task).
Even more..
Turns out that refactoring from relaying
BrokerdFillfrom theopenOrdersub wasn't tested all too well 😂.Fixes a 2nd name error for the
.broker_timefield (which isn't provided in the incremental update msg so this just sets it as thebrokerdtime (fromtime.time()) for now; comments added about how we might approach getting theownTrades(the original value we were passing) in the response message eventually.Also included is a further fix to the paper engine to just avoid trying to pop simulated order book entries on modifies since it's not really necessary and further it was the source of a linger race-crash with the actual paper clear case (popped after already popped by clear task).