Skip to content

[WeeklyReport] EgoistSA 2024.08.10~2024.08.23 - #343

Merged
Harryoung merged 9 commits into
PFCCLab:mainfrom
EgoistSA:EgoistSA/WeeklyReport
Aug 23, 2024
Merged

Harryoung merged 9 commits into
PFCCLab:mainfrom
EgoistSA:EgoistSA/WeeklyReport

Conversation

@EgoistSA

Copy link
Copy Markdown
Contributor

姓名

刘旭(EgoistSA)

开发中的快乐开源任务

这两周我为更多的Tensor API文档增加了图例,并且开始学习怎样为公开API进行标注工作。

本双周工作

  1. 为 Tensor API 文档增加图例

    • 为slice_scatter API文档添加了图例,正在等待合并 #6837
  2. 为公开 API 标注类型提示信息

    • 学习了解为API增加标注的方法
  3. 问题疑惑与解答

    • 问题1:在新增PR时,由于之前的PR还未合并新的PR中也包含了之前commit的文件应该如何处理?

未来双周计划

  1. 为更多API文档增加图例
  2. 尝试为公开 API 标注类型提示信息
  3. 学习文档相关脚本代码

@Harryoung

Copy link
Copy Markdown
Collaborator

同学,针对这个问题「问题1:在新增PR时,由于之前的PR还未合并新的PR中也包含了之前commit的文件应该如何处理?」,一般是新建一个分支,用不同的分支来管理不同的PR哈

@Harryoung Harryoung left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Harryoung
Harryoung merged commit afe8163 into PFCCLab:main Aug 23, 2024
@EgoistSA

Copy link
Copy Markdown
Contributor Author

同学,针对这个问题「问题1:在新增PR时,由于之前的PR还未合并新的PR中也包含了之前commit的文件应该如何处理?」,一般是新建一个分支,用不同的分支来管理不同的PR哈

但是我已经新建分支在fetch upstream之后还是会出现这种情况

@Harryoung

Copy link
Copy Markdown
Collaborator

同学,针对这个问题「问题1:在新增PR时,由于之前的PR还未合并新的PR中也包含了之前commit的文件应该如何处理?」,一般是新建一个分支,用不同的分支来管理不同的PR哈

但是我已经新建分支在fetch upstream之后还是会出现这种情况

检查一下是不是基于干净的主分支来创建的子分支?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants