Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

Conversation

ctkindle
Copy link
Contributor

@ctkindle ctkindle commented Jun 2, 2021

按要求修改后重提的pr

@TCChenlong
Copy link
Contributor

0、标题时间更新一下吧,更新到最新提PR的时间;
1、文章中的介绍不要用"我们",需要用第二人称 "你";
2、第一段的相关介绍可以再提炼一下;
3、使用GPU训练,不需要显式的说明 PLACE = paddle.CUDAPlace(0) # 在gpu上训练;如果是GPU环境,默认会使用GPU进行训练,因此 PLACE = paddle.CUDAPlace(0) 与 PLACE 相关的参数设置都可以删除;
4、注意使用空行提升代码的易读性,如CaltechDatesetmodel.prepare、model.fit 等地方;

@ctkindle
Copy link
Contributor Author

ctkindle commented Jun 7, 2021

0.更新了标题
1.删除了文中"我们"称谓
2.精简了grad-cam的理论介绍
3.删除了指定gpu资源的代码
4.更合理的使用了空行,以增加代码可读性

Copy link
Contributor

@TCChenlong TCChenlong left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants