Skip to content

Conversation

@HumphreyYang
Copy link
Member

Hi @jstac,

This PR addresses our discussions in #105 about the consistent use of math notations.

I have put $\psi$ in the context of $t$ across the lecture since it is more consistent with the plots we have and our discussions in distributions over time.

Could you please review this PR and let me know if there is anything I should change?

Many thanks in advance.

@netlify
Copy link

netlify bot commented Feb 28, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 5e19819
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/63ffe5c193f8140008597342
😎 Deploy Preview https://deploy-preview-108--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

The set $S$ is called the **state space** and $x_1, \ldots, x_n$ are the **state values**.

A **distribution** $\psi$ on $S$ is a probability mass function of length $n$, where $\psi(i)$ is the amount of probability allocated to state $x_i$.
A **distribution** $\psi_t$ on $S$ is a probability mass function of length $n$ at time $t$, where $\psi_t(i)$ is the amount of probability allocated to state $x_i$ at time $t$.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this change --- I think it's irrelevant when we just want to define a distribution.

(This is a judgement call on the trade-off between constency and simplicity.)

@jstac
Copy link
Contributor

jstac commented Feb 28, 2023

Many thanks @HumphreyYang , the lecture is clearer now. Just one small change above.

@github-actions github-actions bot temporarily deployed to commit February 28, 2023 03:58 Inactive
@HumphreyYang
Copy link
Member Author

Many thanks for your review @jstac. I have reverted this change :)

@HumphreyYang HumphreyYang requested a review from jstac February 28, 2023 22:35
@HumphreyYang
Copy link
Member Author

HumphreyYang commented Mar 1, 2023

Many thanks @jstac for the detailed read. I have updated the draft. One place I am not sure whether to use $\psi_0$ and $\psi_0$ is here. I think it we should leave it as it is now, but a quick confirmation would be greatly appreciated :)

@HumphreyYang
Copy link
Member Author

Hi @mmcky, there seems to be a cache error similar to what we had before:

Exception occurred:
  File "/usr/share/miniconda3/envs/quantecon/lib/python3.9/site-packages/jupyter_cache/cache/db.py", line 57, in session_context
    raise RuntimeError(
RuntimeError: Unexpected error accessing jupyter cache, it may need to be cleared.
The full traceback has been saved in /tmp/sphinx-err-snkrx4oe.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
Traceback (most recent call last):
  File "/usr/share/miniconda3/envs/quantecon/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1819, in _execute_context
    self.dialect.do_execute(
  File "/usr/share/miniconda3/envs/quantecon/lib/python3.9/site-packages/sqlalchemy/engine/default.py", line 732, in do_execute
    cursor.execute(statement, parameters)
sqlite3.OperationalError: no such table: nbproject

Could you please have a look to see if the cache needs to be cleaned?

Many thanks in advance.

@jstac
Copy link
Contributor

jstac commented Mar 1, 2023

Many thanks @jstac for the detailed read. I have updated the draft. One place I am not sure whether to use ψ0 and ψ0 is here. I think it we should leave it as it is now, but a quick confirmation would be greatly appreciated :)

I think that's fine as is @HumphreyYang . Many thanks.

@mmcky please merge when ready.

@github-actions github-actions bot temporarily deployed to commit March 1, 2023 23:58 Inactive
@mmcky mmcky merged commit 51b2d5b into main Mar 2, 2023
@mmcky mmcky deleted the mc_math_notation branch March 2, 2023 01:11
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.

4 participants