QuantEcon / lecture-python

Source files for https://python.quantecon.org
BSD 3-Clause "New" or "Revised" License
61 stars 21 forks source link

Error in "Asset Pricing with Incomplete Markets" lecture #373

Closed jms202 closed 3 years ago

jms202 commented 3 years ago

The matrix immediately following the text "Temporarily pessimistic believe the transition matrix" should be:

[[2/3, 1/3], [2/3, 1/3]]

as it is in the solution to the exercise

jstac commented 3 years ago

Thanks @jms202 . @sayaikegawa or @shlff, could you please review this when you have time and submit a PR? (Unless you would like to submit a PR @jms202 ?)

Also, the word "investor" should be inserted before "pessimistic" in the line quoted above.

shlff commented 3 years ago

Thanks @jstac , @jms202 and @sayaikegawa . The merged PR #368 had already fixed this issue.

jstac commented 3 years ago

Thanks @shlff .

@jms202, are you using the PDF version? If so, is it the most recent?

jms202 commented 3 years ago

No I'm using the web version: https://python.quantecon.org/harrison_kreps.html.

The matrices P_o and P_p are still appearing the same for me.

On Tue, 23 Feb 2021, 00:22 John Stachurski, notifications@github.com wrote:

Thanks @shlff https://github.com/shlff .

@jms202 https://github.com/jms202, are you using the PDF version? If so, is it the most recent?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/QuantEcon/lecture-python/issues/373#issuecomment-783775523, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADYMWEIPUCSGC5HYAUWBAVTTALYNDANCNFSM4YAJWLAA .

jstac commented 3 years ago

@mmcky , can you please tell me if the merge has been made live?

mmcky commented 3 years ago

@jstac I just reviewed the last set of changes on the preview server and these changes (which includes #368) and they are now live.

jstac commented 3 years ago

Thanks @mmcky . Hopefully that fixes your issue @jms202 . (Might need to refresh browser.) If not please reopen.