slipher
slipher
LGTM to `renderer: iterate shader stage array directly, avoid array of pointers` and `tr_shader: also list shaders without stages` I have not reviewed the other one.
Disabling the ckit is not really compatible with standard gameplay, so I feel this is not a case bot developers should have to deal with. The juggernaut mod has a...
On the other hand we *should* handle `g_bot_ckit` being disabled. So if we did that using an API for equipment availability, then the case of ckit being disabled for everyone...
> Did anything depend on the old behavior? Only the bugs :)
> Which bugs exactly? This one is the biggest: `Resetting myTimer fixes a bug pointed out by Sweet: if a bot switches to our default behavior tree from any other...
I would definitely consider that a bug in the framework more than a bug in the individual BTs. It's absurd to require every BT that uses a timer to have...
> I agree that resetting the bot states is a good thing. But I am not yet convinced about some fields. Should we reset the planned path completely? Or should...
TL;DR I removed code to clear `lastNavcon*` on behavior change. To clarify things, `currentNode`, `runningNodes`, `goal`, and `m_nav` are all cleared on behavior change on master. The original version of...
> I'm just brainstorming. Could we really make the bot go the next path corner? Maybe. Seems doable. I think the way to do this would be to keep the...