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.

Sign up for GitHub

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

Jump to bottom

[PaddlePaddle Hackathon] add paddle.vision.models.InceptionV3 #36064

Merged
merged 2 commits into from
Oct 22, 2021
Merged

[PaddlePaddle Hackathon] add paddle.vision.models.InceptionV3 #36064

merged 2 commits into from
Oct 22, 2021

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Sep 24, 2021 •
edited
Loading

PR types

New features

PR changes

APIs

Describe

status: Merged

Performance

AI Studio 测试详情：https://aistudio.baidu.com/aistudio/projectdetail/2420319?contributionType=1
基准参考：https://github.com/PaddlePaddle/PaddleClas/blob/release/2.2/docs/en/ImageNet_models_en.md

Model Top-1 Top-5
InceptionV3 0.7906(-0.0008) 0.9450(-0.0009)

括号中为以 PaddleClas 性能基准为参考的偏差值

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@SigureMo SigureMo mentioned this pull request Sep 24, 2021
Closed
@SigureMo SigureMo mentioned this pull request Sep 24, 2021
Merged
@SigureMo SigureMo marked this pull request as ready for review September 24, 2021 16:11
@SigureMo SigureMo changed the title add Inceptionv3 model [PaddlePaddle Hackathon] add InceptionV3 Sep 26, 2021
@SigureMo SigureMo force-pushed the inceptionv3 branch 2 times, most recently from 6af4ffe to d047fcf Compare September 29, 2021 15:41
@SigureMo SigureMo mentioned this pull request Oct 8, 2021
Merged
3 tasks
LielinJiang
LielinJiang previously approved these changes Oct 13, 2021
View reviewed changes
TCChenlong
TCChenlong reviewed Oct 20, 2021
View reviewed changes
python/paddle/vision/models/inceptionv3.py
return x


class InceptionV3(nn.Layer):
Copy link
Contributor

Choose a reason for hiding this comment

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

英文文档的代码示例应与中文一致~

python/paddle/vision/models/inceptionv3.py
Args:
pretrained (bool): If True, returns a model pre-trained on ImageNet

Examples:
Copy link
Contributor

Choose a reason for hiding this comment

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

同上

Copy link
Member Author

Choose a reason for hiding this comment

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

已修改～～～

SigureMo and others added 2 commits October 20, 2021 14:35
@SigureMo @Ainavo @pithygit
add inceptionv3
905eb45 
* add unit test
* update API name
* sort imports
* class_num -> num_classes
* add code example
* export to paddle.vision
* add pretrained test
* add with_pool parameter

Co-authored-by: Ainavo <ainavo@163.com>
Co-authored-by: pithygit <pyg20200403@163.com>
@SigureMo
update en docs
b5c1623
TCChenlong
TCChenlong approved these changes Oct 21, 2021
View reviewed changes
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

XiaoguangHu01
XiaoguangHu01 approved these changes Oct 21, 2021
View reviewed changes
Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

@LielinJiang LielinJiang merged commit ff06df6 into PaddlePaddle:develop Oct 22, 2021
@SigureMo SigureMo deleted the inceptionv3 branch October 22, 2021 03:19
@TCChenlong TCChenlong mentioned this pull request Oct 22, 2021
Merged
@TCChenlong TCChenlong changed the title [PaddlePaddle Hackathon] add InceptionV3 [PaddlePaddle Hackathon] paddle.vision.models.InceptionV3 Mar 15, 2022
@TCChenlong TCChenlong changed the title [PaddlePaddle Hackathon] paddle.vision.models.InceptionV3 [PaddlePaddle Hackathon] add paddle.vision.models.InceptionV3 Mar 15, 2022
@SigureMo SigureMo mentioned this pull request Sep 7, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LielinJiang LielinJiang LielinJiang approved these changes

@XiaoguangHu01 XiaoguangHu01 XiaoguangHu01 approved these changes

@TCChenlong TCChenlong TCChenlong approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@SigureMo @TCChenlong @XiaoguangHu01 @LielinJiang