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

Button press reward functions also reward pulling on button #389

Closed
krzentner opened this issue Jan 26, 2023 · 2 comments
Closed

Button press reward functions also reward pulling on button #389

krzentner opened this issue Jan 26, 2023 · 2 comments
Assignees

Comments

@krzentner
Copy link
Contributor

This rarely matters, but button-push family of tasks also reward pulling on the button. This makes it possible to get very high reward without ever succeeding at the task, which should probably be fixed.

Fortunately this rarely matters in practice, since most RL algorithms never attempt to pull on the button.

@reginald-mclean reginald-mclean self-assigned this Feb 2, 2023
@reginald-mclean
Copy link
Collaborator

@krzentner any ideas on ways to fix this?

@krzentner
Copy link
Contributor Author

I saw this happening in one of my experiments, but it's not obvious looking at the code what is wrong. The first step to fixing it would probably be reproducing the issue (perhaps by making a modified scripted policy that triggers the bug). The relevant line of the reward function is here.

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

No branches or pull requests

2 participants