Codecov Report

❌ Patch coverage is 21.62162% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.13%. Comparing base (f458350) to head (d3e09cd).
⚠️ Report is 241 commits behind head on master.

@@            Coverage Diff             @@
##           master     #780      +/-   ##
==========================================
- Coverage   83.38%   83.13%   -0.25%     
==========================================
  Files          39       47       +8     
  Lines        3918     5628    +1710     
==========================================
+ Hits         3267     4679    +1412     
- Misses        651      949     +298     

@hsahovic this is ready for review whenever you get a chance!

I could be convinced this makes sense, but my understanding is that this is trying to rely on choose_move for teampreview, which I'm not convinced is correct. Why do you think we should generally do it that way?

@hsahovic I'm doing that because choose_move is gathering decisions from the _EnvPlayers, which is the action input from env.step(). Basically this allows the RL agent to control the battle during teampreview using switch actions. One could do it differently I suppose but I see no reason why not to do it like this, since the switch action translates 1-to-1 to the teampreview decision case (you just need to make two double-switch decisions in a row to get a teampreview decision of length 4).