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

Fix no framework proto file #5717

Merged
merged 1 commit into from
Nov 17, 2017
Merged

Fix no framework proto file #5717

merged 1 commit into from
Nov 17, 2017

Conversation

Yancey1989
Copy link
Contributor

Fixed #5698

tensor-tang
tensor-tang approved these changes Nov 17, 2017
View reviewed changes
Copy link
Contributor

@tensor-tang tensor-tang left a comment

Choose a reason for hiding this comment

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

I have tried locally, and it works.

@tensor-tang tensor-tang merged commit 00a5155 into PaddlePaddle:develop Nov 17, 2017
@Yancey1989 Yancey1989 deleted the fix_framework_proto_error branch November 17, 2017 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tensor-tang tensor-tang tensor-tang approved these changes

@luotao1 luotao1 Awaiting requested review from luotao1

@qingqing01 qingqing01 Awaiting requested review from qingqing01

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@Yancey1989 @tensor-tang