Skip to content

Conversation

@sudayuga
Copy link
Contributor

Fixed inconsistencies between the lecture description and the code.

@netlify
Copy link

netlify bot commented Oct 27, 2025

Deploy Preview for lustrous-melomakarona-3ee73e ready!

Name Link
🔨 Latest commit d3845d4
🔍 Latest deploy log https://app.netlify.com/projects/lustrous-melomakarona-3ee73e/deploys/6902f334d94c5b00081ae548
😎 Deploy Preview https://deploy-preview-288--lustrous-melomakarona-3ee73e.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 project configuration.

@mmcky mmcky requested a review from Copilot October 30, 2025 03:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR corrects three mathematical errors in the LQ Ramsey optimal taxation lecture, fixing formulas in both the mathematical exposition and the computational implementation.

  • Removed extraneous exponent j from the risk-free rate definition
  • Corrected summation index from t to s in cumulative Pi definition
  • Fixed sign error in quadratic form computation from subtraction to addition

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mmcky mmcky requested a review from HumphreyYang October 30, 2025 03:48
@HumphreyYang HumphreyYang changed the title Fix calculation of H in lqramsey.md [lqramsey] Fix calculation of H Oct 30, 2025
@HumphreyYang
Copy link
Member

Many thanks @sudayuga for picking out the typos!

Hi @mmcky, it looks great to me!

@mmcky
Copy link
Contributor

mmcky commented Oct 30, 2025

thanks @sudayuga and @HumphreyYang

@mmcky mmcky merged commit bc9f017 into QuantEcon:main Oct 30, 2025
8 checks passed
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.

3 participants