Closed maurerle closed 1 year ago
Attention: 3 lines
in your changes are missing coverage. Please review.
Comparison is base (
606c335
) 78.03% compared to head (7482680
) 78.10%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I hardly get what is happening here. Nice though if it works. I am looking forward to next development meeting :)
Weird that get_sum_reward
did not have a length and I needed to handle that case?
This does not occur on my device though.
This includes fixes to running the MQTT container with docker for a distributed simulation.
For distributed simulations, one can start
python examples/distributed_world_manager.py
in one terminal and runpython examples/distributed_world_agent.py
in another one.A registry of which agents are available (and must be online before simulation can start) and a central yaml config are still needed. So this is an early fix set, which should not change anything to existing behavior