Skip to content

[BUG FIX] Fix pickling of 'reward_cfg["reward_scales"]' in hover example#1340

Merged
duburcqa merged 1 commit intoGenesis-Embodied-AI:mainfrom
ZaberKo:fix-hover-example
Jul 1, 2025
Merged

[BUG FIX] Fix pickling of 'reward_cfg["reward_scales"]' in hover example#1340
duburcqa merged 1 commit intoGenesis-Embodied-AI:mainfrom
ZaberKo:fix-hover-example

Conversation

@ZaberKo
Copy link
Contributor

@ZaberKo ZaberKo commented Jul 1, 2025

Description

Currently, HoverEnv.reward_scales is a shadow reference of reward_cfg["reward_scales"], so later modification on self.reward_scales will affect reward_cfg["reward_scales"], leading to wrong reward_cfg in cfgs.pkl.

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@duburcqa duburcqa enabled auto-merge (squash) July 1, 2025 08:39
@duburcqa duburcqa changed the title [BUG FIX] Fix wrong reward_cfg["reward_scales"] in cfgs.pkl for hover example [BUG FIX] Fix pickling of 'reward_cfg["reward_scales"]' in hover example Jul 1, 2025
@duburcqa duburcqa merged commit 34115dc into Genesis-Embodied-AI:main Jul 1, 2025
24 of 25 checks passed
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.

2 participants