refactored multi agent proposal

[Mytolo]

Instead of returning a generator for RLBase.plan! use NamedTuples s.t. the environment is able to gather the actions for the players correctly. It might be that the iteration of the environment is not in order (for example, this might occur when it is wrapped from another language into julia).

Moreover, this PR includes proper push operations for the Basic Agent policy for simultaneous dynamicStyle trait envs. They were missing. Hence, the trajectory was not updated acordingly.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 18 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (1f7f347) to head (ec34889).
Report is 101 commits behind head on main.

Files with missing lines	Patch %	Lines
...inforcementLearningCore/src/policies/agent/base.jl	0.00%	12 Missing ⚠️
...mentLearningCore/src/policies/agent/multi_agent.jl	0.00%	6 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (1f7f347) and HEAD (ec34889). Click for more details.

HEAD has 4 uploads less than BASE

Flag	BASE (1f7f347)	HEAD (ec34889)
	5	1

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #926       +/-   ##
==========================================
- Coverage   13.64%   0.00%   -13.65%     
==========================================
  Files         217     208        -9     
  Lines        7571    7415      -156     
==========================================
- Hits         1033       0     -1033     
- Misses       6538    7415      +877     

Files with missing lines	Coverage Δ
...mentLearningCore/src/policies/agent/multi_agent.jl	0.00% <0.00%> (ø)
...inforcementLearningCore/src/policies/agent/base.jl	0.00% <0.00%> (ø)

... and 42 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Mytolo]

NOTE: CI fail depends on rework of run loop as RLBase.plan! is executed after termination of environment as stated in #913, PR #921

[HenriDeh]

Doesn't the new loop work for you ?