Skip to content

Update recommendation system book test#11252

Closed
nickyfantasy wants to merge 2 commits intoPaddlePaddle:developfrom
nickyfantasy:update_recommendation_system_chapter
Closed

Update recommendation system book test#11252
nickyfantasy wants to merge 2 commits intoPaddlePaddle:developfrom
nickyfantasy:update_recommendation_system_chapter

Conversation

@nickyfantasy
Copy link
Contributor

@nickyfantasy nickyfantasy commented Jun 6, 2018

-Move test reader outside
-Set trainer stop condition as 5 steps
-Compare Predict rating and actual rating

-Set trainer stop condition as 5 steps
-Compare Predict rating and actual rating
@nickyfantasy nickyfantasy changed the title -Move test_reader outside Jun 6, 2018
sidgoyal78
sidgoyal78 previously approved these changes Jun 6, 2018
Copy link
Contributor

@sidgoyal78 sidgoyal78 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Contributor

@sidgoyal78 sidgoyal78 left a comment

Choose a reason for hiding this comment

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

LGTM

@paddle-bot-old paddle-bot-old bot closed this May 22, 2020
@paddle-bot-old
Copy link

Since you haven't replied for a long time, we have closed this issue/pr.
If the problem is not solved or there is a follow-up one, please reopen it at any time and we will continue to follow up.
由于您长期未回复,我们将关闭这个issue/pr。
若问题未解决或有后续问题,请随时重新打开,我们会继续跟进。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants