-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
[bug] Fix obvious bugs across all lectures #95
Conversation
✅ Deploy Preview for playful-platypus-17d3bb ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @HumphreyYang - thank you. I'd like to get this merged as it touches a lot of files. Does it address any of the issues (i.e. titles) -- just be nice to link them if it does. Thanks.
@@ -366,7 +366,7 @@ $$ | |||
以及 | |||
|
|||
$$ | |||
\Sigma_y = \tilde A (\σ_u^2 I_{T \times T} ) \tilde A^T | |||
\Sigma_y = \tilde A (\sigma_u^2 I_{T \times T} ) \tilde A^T |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@HumphreyYang should these be unicode sigma?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've just left this as a question.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks @mmcky, I think this is correct under the latex environment: )
@HumphreyYang I see the fixes lines now. Thanks. |
Hi @mmcky,
I spent some time hovering over all the lectures and fixing obvious bugs I spotted.
Please kindly merge this if it looks good.
Fixes #62
Fixes #73
Fixes #65