Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature(wrh): Add Harmony Dream loss balance in MuZero #242

Merged
merged 6 commits into from
Jul 16, 2024

Conversation

ruiheng123
Copy link
Contributor

@ruiheng123 ruiheng123 commented Jul 8, 2024

Harmony Dream: Dynamically balance the weights of different loss functions during training.
For more details, refer to original paper: https://arxiv.org/abs/2310.00344

@puyuan1996 puyuan1996 added the enhancement New feature or request label Jul 8, 2024
lzero/model/muzero_model.py Outdated Show resolved Hide resolved
lzero/policy/muzero.py Show resolved Hide resolved
lzero/model/muzero_model.py Outdated Show resolved Hide resolved
@puyuan1996 puyuan1996 changed the title feature(wrh): Add Harmony Dream in Muzero feature(wrh): Add Harmony Dream loss balance in MuZero Jul 8, 2024
zoo/atari/config/atari_muzero_config.py Outdated Show resolved Hide resolved
torch.log(torch.exp(self.harmony_policy) + 1) +
torch.log(torch.exp(self.harmony_value) + 1) +
torch.log(torch.exp(self.harmony_reward) + 1)
# torch.log(torch.exp(self.harmony_entropy) + 1)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

把549,553,557行的harmony_dynamics和harmony_entropy相关的注释删除一下

lzero/policy/muzero.py Show resolved Hide resolved
lzero/policy/muzero.py Outdated Show resolved Hide resolved
@puyuan1996 puyuan1996 added the polish Polish algorithms, tests or configs label Jul 16, 2024
@puyuan1996 puyuan1996 merged commit 032825f into opendilab:main Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request polish Polish algorithms, tests or configs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants