Skip to content

[AR1] Update editorial suggestions #458

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

Merged
merged 5 commits into from
Jul 8, 2024
Merged

[AR1] Update editorial suggestions #458

merged 5 commits into from
Jul 8, 2024

Conversation

SylviaZhaooo
Copy link
Collaborator

@SylviaZhaooo SylviaZhaooo commented Jun 10, 2024

Dear Professor Stachurski @jstac,

I have revised the lecture AR(1) Processes according to issue #451.

Additionally, since the concept of moment has not been introduced specifically in the lecture LLN and CLT, I have directly hyperlinked the term to the Wikipedia page for this concept, consistent with its use in the lecture LLN and CLT.

I have also used the economy's growth rate as an example of ergodicity and highlighted when it fails.

Best Regards,
Jingkun

Copy link

netlify bot commented Jun 10, 2024

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit a766c9c
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/668b47b521664a0008bd5531
😎 Deploy Preview https://deploy-preview-458--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 configuration.

Copy link

github-actions bot commented Jun 10, 2024

@github-actions github-actions bot temporarily deployed to pull request June 10, 2024 03:19 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 10, 2024 03:20 Inactive
@SylviaZhaooo SylviaZhaooo requested a review from mmcky June 10, 2024 03:20
@jstac
Copy link
Contributor

jstac commented Jun 10, 2024

Thanks @SylviaZhaooo , much appreciated. I'll add some small edits before I merge this.

@mmcky
Copy link
Contributor

mmcky commented Jun 28, 2024

@jstac just had a minor comment. I will leave this with you re: final edits (as per comment above)

@github-actions github-actions bot temporarily deployed to pull request June 30, 2024 07:24 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 30, 2024 07:25 Inactive
@mmcky mmcky requested review from mmcky and jstac June 30, 2024 23:42
@mmcky mmcky added the ready label Jul 1, 2024
@jstac jstac added in-work and removed ready labels Jul 1, 2024
@jstac
Copy link
Contributor

jstac commented Jul 8, 2024

@mmcky , I seem to have generated a conflict here by failing to pull some changes. With apologies, would you mind to look into this? Hopefully it's an easy fix.

@mmcky
Copy link
Contributor

mmcky commented Jul 8, 2024

@mmcky , I seem to have generated a conflict here by failing to pull some changes. With apologies, would you mind to look into this? Hopefully it's an easy fix.

@jstac this is now fixed and up to date. 👍

@github-actions github-actions bot temporarily deployed to pull request July 8, 2024 02:04 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 8, 2024 02:05 Inactive
@jstac
Copy link
Contributor

jstac commented Jul 8, 2024

Nice work, many thanks @mmcky .

@jstac jstac merged commit fbe4c82 into main Jul 8, 2024
8 checks passed
@jstac jstac deleted the update_AR1 branch July 8, 2024 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants