Skip to content

Fix sweep match metadata parsing#593

Open
devshahofficial wants to merge 1 commit into
PufferAI:4.0from
devshahofficial:devshahofficial/fix-sweep-match-metadata
Open

Fix sweep match metadata parsing#593
devshahofficial wants to merge 1 commit into
PufferAI:4.0from
devshahofficial:devshahofficial/fix-sweep-match-metadata

Conversation

@devshahofficial

Copy link
Copy Markdown

Summary

  • Treat self-play match sweep settings as sweep metadata instead of hyperparameter definitions.
  • Add a config regression covering the default + breakout sweep config path.

Root cause

The match sweep keys added under [sweep] are scalar settings. _params_from_puffer_sweep expected every non-metadata key in that section to be a hyperparameter dict, so puffer sweep breakout raised AssertionError: Param match_enemy_model_path is not a dict.

Validation

  • python -m pytest tests/test_sweep_config.py tests/test_import_performance.py -q
  • Direct parse of config/default.ini + config/breakout.ini through _params_from_puffer_sweep

Fixes #589

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.

Default sweep broken for most envs

1 participant