Skip to content
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

【Hackathon 4 No.86】【开源社区对比研究】 以贡献文档为例 #420

Merged
merged 6 commits into from
Apr 18, 2023

Conversation

gouzil
Copy link
Member

@gouzil gouzil commented Mar 6, 2023

@paddle-bot
Copy link

paddle-bot bot commented Mar 6, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请检查PR提交格式和内容是否完备,具体请参考示例模版
Your PR has been submitted. Thanks for your contribution!
Please check its format and content. For this, you can refer to Template and Demo.

Copy link
Collaborator

@jzhang533 jzhang533 left a comment

Choose a reason for hiding this comment

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

  • 丰富程度上,可以展开跟pytorch的github内容进行对比,这部分我直觉上paddle是差的比较多的。
  • 文档组织上,看是否可以用两三个实际的案例(比如寻找完成某个开发任务所需要的文档),来分析一下现在的文档组织上,是否有不合理的地方。
  • 更新及时性,不止局限于git log所看到的commit频次,可以再深入分析一下。

另外,总结部分可以写的更详细一些。

@gouzil
Copy link
Member Author

gouzil commented Mar 7, 2023

  • 丰富程度上,可以展开跟pytorch的github内容进行对比,这部分我直觉上paddle是差的比较多的。
  • 文档组织上,看是否可以用两三个实际的案例(比如寻找完成某个开发任务所需要的文档),来分析一下现在的文档组织上,是否有不合理的地方。
  • 更新及时性,不止局限于git log所看到的commit频次,可以再深入分析一下。

另外,总结部分可以写的更详细一些。

老实说,看了这么多贡献文档。总感觉 paddle 在内容上还是太少了。

可以试试让研发和 pfcc 来补充一些更加深入一些的内容,不再局限于算子或api,也可以是一些设计思路,哪怕是 paddle 的实验性功能或者下一个发展方向也是可以的。这样说不定也能吸引一些不同领域的开发者来参与共建

@jzhang533
Copy link
Collaborator

老实说,看了这么多贡献文档。总感觉 paddle 在内容上还是太少了。
嗯,我也觉得太少了。

Copy link
Collaborator

@jzhang533 jzhang533 left a comment

Choose a reason for hiding this comment

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

PR将会合入。
根据黑客松活动安排,请等待通知进行线上的报告分享。
在这之前,可以继续丰富报告。

@jzhang533 jzhang533 merged commit c03f957 into PaddlePaddle:master Apr 18, 2023
@gouzil
Copy link
Member Author

gouzil commented Apr 18, 2023

PR将会合入。

根据黑客松活动安排,请等待通知进行线上的报告分享。

在这之前,可以继续丰富报告。

收到

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.

None yet

4 participants