Skip to content

[mccall_model] Translation update #17

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 7 commits into from
Apr 24, 2025
Merged

Conversation

nisha617
Copy link
Collaborator

Hi @HumphreyYang ,

McCall model is updated now. Please have a look.

Copy link

netlify bot commented Mar 25, 2025

Deploy Preview for astonishing-narwhal-a8fc64 ready!

Name Link
🔨 Latest commit a56399a
🔍 Latest deploy log https://app.netlify.com/sites/astonishing-narwhal-a8fc64/deploys/680ac1d93c9c7a0008f6f75b
😎 Deploy Preview https://deploy-preview-17--astonishing-narwhal-a8fc64.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.

@HumphreyYang HumphreyYang changed the title [McCall Model] Translation update [mccall_model] Translation update Mar 25, 2025
@HumphreyYang HumphreyYang requested a review from Copilot March 26, 2025 23:30
Copy link

@Copilot 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 updates the translation of the McCall model lecture, refining wording and terminology for clarity in the Chinese version. Key changes include improved wording of theoretical explanations, enhanced consistency in bullet lists and phrasing, and minor adjustments in technical descriptions.

Comments suppressed due to low confidence (2)

lectures/mccall_model.md:356

  • [nitpick] Consider using consistent terminology for representing choices throughout the document. If other sections use '状态-动作对' or 'action' is translated differently elsewhere, ensure that the same term is uniformly applied.
这是一个用于存储数据并计算状态-行动对的值的类,即基于当前状态和任意可行的行动,计算贝尔曼方程 {eq}`odu_pv2p` 右侧最大值括号中的值。

lectures/mccall_model.md:785

  • [nitpick] Verify that the term '等值线图' is consistent with the rest of the documentation. '等高线图' is a common translation for contour plots and might be preferred for clarity.
我们将使用等值线图来分析这个问题。

Copy link

github-actions bot commented Mar 28, 2025

@github-actions github-actions bot temporarily deployed to pull request March 28, 2025 03:12 Inactive
@HumphreyYang HumphreyYang merged commit 8012bf3 into main Apr 24, 2025
1 check passed
@HumphreyYang HumphreyYang deleted the update-mccall_model.md branch April 24, 2025 22:57
@github-actions github-actions bot temporarily deployed to pull request April 24, 2025 23:07 Inactive
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