Skip to content

Extra logging#37

Open
icunnyngham wants to merge 5 commits intoAechPro:mainfrom
icunnyngham:extra_logging
Open

Extra logging#37
icunnyngham wants to merge 5 commits intoAechPro:mainfrom
icunnyngham:extra_logging

Conversation

@icunnyngham
Copy link
Contributor

Environment specific logging which can be saved to CSVs or to wandb via redis when using DistribPolicyGradients.

Note:

  • I put the worker-side aggregation step into the MARL agent because I needed access to cumulative timesteps right before experience is sent. If this could be achieved more cleanly, keeping extra_logger confined to DistribPolicyGradients, I would be happy to change this
  • All the places where i had to add extra_logger as an argument for build_rlgym_from_config are pretty ugly, but I wasn't sure how else to achieve this. That function needs access to the logger to inject it into the rewards and obs functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant