Conversation
refix the ncigs check
|
I'm finally reading through this and I think the vast majority of it is good, a bunch of good changes. We've really struggled with this stuff for a while, and I like this approach. However, the way it's tracked on What do you think of instead storing maps in the Then for I'd love to hear your thoughts on this. |
|
hmm - so I think the worst case is ~60-100mb across 30-50 games, but that is a lot of to be adding that we don't need to be, yeah. I think the map approach will be good, and will still be 1000x faster than diffing was, yeah. I'll give it a go over easter. |
|
The log diffs should be based on the |
|
yeah - currently I haven't needed to make any changes to the frontend or replays because it ends up invisible to both of them |
|
Alright, I was able to update the log system so that the messages are maps of {:public .. :corp .. :runner ..} as a single list. |
|
That's incredible, thank you. Merge when ready, we can fix whatever bugs have snuck in after we get some debugging in. |
Builds on #8630
This splits the logs into three seperate logs.
Replays and spectators only see the public logs.
The corp and runner only see their side specific logs (along with side-specific spectators).
system-msg can take a side, or list of sides, as things to print to.
You can also do
multi-msg state side [map].:publicbackfills into empty:corpor:runner.:msgmaps can optional be a map of{:public msg :corp msg :runner msg}. Public backfills into empty corp or runner.Some examples:


Closes #8628
My goal for this PR is that this works for basically everything in standard (excepting costs right now).
I've gone through the following so far to take advantage of this: