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

Ex4.7-A possible bug #82

Open
khbalhandawi opened this issue Apr 21, 2021 · 2 comments
Open

Ex4.7-A possible bug #82

khbalhandawi opened this issue Apr 21, 2021 · 2 comments

Comments

@khbalhandawi
Copy link

In your python file Ex4.7-A.py line 51 I think it should read

temp[((value_A_Changed, value_B_Changed),reward)] = temp.get( ((value_A_Changed, value_B_Changed),reward), 0 )

instead of

temp[((value_A_Changed, value_B_Changed),reward)] = temp.get( (value_A_Changed, value_B_Changed), 0 )

The second line above will always return 0 because the key (value_A_Changed, value_B_Changed) does not exist in temp
I tried rerunning it with this change and could not reproduce the answer of the book. I am attaching the optimal policy map that I got

pi_4

@wqp89324
Copy link

wqp89324 commented Sep 3, 2021

I also have a question about the original car rental company problem, what if location A only has 2 cars, but a random policy asks to move 5 cars from location A to B?

@LiuZhihao2022
Copy link

I also come across this question. It will lead to zero probability if reward is not added to the key.

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

3 participants