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

change some dygraph models to use data loader #4595

Merged
merged 3 commits into from
May 28, 2020
Merged

change some dygraph models to use data loader #4595

merged 3 commits into from
May 28, 2020

Conversation

chenwhql
Copy link
Contributor

@chenwhql chenwhql commented May 7, 2020 •
edited
Loading

change 4 dygraph models to use DataLoader to read data:

  • mnist
  • resnet
  • se_resnet
  • ptb_lm

polish DataLoader writing:

  • mobilenet

@chenwhql chenwhql requested review from phlrain and zhiqiu May 8, 2020 02:47
zhiqiu
zhiqiu reviewed May 9, 2020
View reviewed changes
Copy link
Contributor

@zhiqiu zhiqiu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM, by the way, would you plz add some experimental data of performance.

@chenwhql
Copy link
Contributor Author

chenwhql commented May 9, 2020

LGTM, by the way, would you plz add some experimental data of performance.

refer to #4376, 单进程数据

@phlrain phlrain merged commit b0239e3 into PaddlePaddle:develop May 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhiqiu zhiqiu zhiqiu left review comments

@phlrain phlrain phlrain 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.

None yet
3 participants
@chenwhql @zhiqiu @phlrain