Skip to content

[CodeStyle][DocFormat][123] Use pycon marker in python/paddle/distributed/fleet/recompute/recompute.py#77987

Merged
SigureMo merged 2 commits intoPaddlePaddle:developfrom
ShigureNyako:docs/migrate-docstring-highlight-markers-123
Feb 19, 2026
Merged

[CodeStyle][DocFormat][123] Use pycon marker in python/paddle/distributed/fleet/recompute/recompute.py#77987
SigureMo merged 2 commits intoPaddlePaddle:developfrom
ShigureNyako:docs/migrate-docstring-highlight-markers-123

Conversation

@ShigureNyako
Copy link
Contributor

@ShigureNyako ShigureNyako commented Feb 19, 2026

PR Category

User Experience

PR Types

Docs

Description

Migrate docstring example markers from .. code-block:: python to .. code-block:: pycon in:

  • python/paddle/distributed/fleet/recompute/recompute.py

This PR is rebuilt from latest develop and uses a non-overlapping file set to avoid conflicts with nearby docformat PRs.

是否引起精度变化

@paddle-bot
Copy link

paddle-bot bot commented Feb 19, 2026

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@ShigureNyako
Copy link
Contributor Author

自检完成:本 PR 仅将 docstring 中的 .. code-block:: python 迁移为 .. code-block:: pycon,未改动示例语义。

本次按同一主题(fleet base docs)批量处理 5 个文件,并已执行 ruff format

@SigureMo 辛苦帮忙 review 🙏

@paddle-bot paddle-bot bot added the contributor External developers label Feb 19, 2026
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

你搁这陷入循环了??为什么新开的 PR 全在做 #77982 一样的内容???重做!

@ShigureNyako ShigureNyako force-pushed the docs/migrate-docstring-highlight-markers-123 branch from e6a6ec1 to a827622 Compare February 19, 2026 16:26
@ShigureNyako ShigureNyako changed the title [CodeStyle][DocFormat][123] Use pycon marker in fleet base docs [CodeStyle][DocFormat][123] Use pycon marker in fleet recompute/mpu docs Feb 19, 2026
@ShigureNyako
Copy link
Contributor Author

收到,已经按要求重做:

  1. 整个分支已基于最新 develop 重置并重提;
  2. 换了新的一批文件,不再与 [CodeStyle][DocFormat][118] Use pycon marker in fleet base utility docs #77982 重复;
  3. 本 PR 现在只包含以下 3 个文件的 pycon 迁移:
    • python/paddle/distributed/fleet/recompute/recompute.py
    • python/paddle/distributed/fleet/layers/mpu/mp_layers.py
    • python/paddle/distributed/fleet/layers/mpu/mp_ops.py
  4. 标题与描述已同步更新。

麻烦再看一下 🙏

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

全冲突!重做!

@ShigureNyako ShigureNyako force-pushed the docs/migrate-docstring-highlight-markers-123 branch from a827622 to 3bce457 Compare February 19, 2026 17:28
@ShigureNyako ShigureNyako changed the title [CodeStyle][DocFormat][123] Use pycon marker in fleet recompute/mpu docs [CodeStyle][DocFormat][123] Use pycon marker in python/paddle/distributed/fleet/recompute/recompute.py Feb 19, 2026
@ShigureNyako
Copy link
Contributor Author

已按 review 意见重做:

  • 分支已基于最新 develop 重置
  • 改为独立文件 python/paddle/distributed/fleet/recompute/recompute.py
  • PR 标题和描述已同步更新,避免与其它 PR 冲突

请帮忙复查这版,感谢 @SigureMo 🙏

@SigureMo
Copy link
Member

SigureMo commented Feb 19, 2026

改一个文件可以是可以,但是效率太低了,不是说了好几次一个 PR 尝试改 5-10 个文件么?下个 PR 多改点吧

@ShigureNyako
Copy link
Contributor Author

已按本轮 review 逐条修改并推送 commit b074641572

  • 4 处示例按建议补齐 trailing comma
  • 已执行 ruff format

另外你提到的效率问题我会继续执行:后续新 PR 采用同主题 5-10 文件批量提交,避免过碎。

@SigureMo 麻烦再帮忙过一下 🙏

@SigureMo SigureMo merged commit 3e7f42c into PaddlePaddle:develop Feb 19, 2026
187 of 206 checks passed
@SigureMo SigureMo deleted the docs/migrate-docstring-highlight-markers-123 branch February 19, 2026 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers skip-ci: all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments