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

[High-Level-API]Compare predict and actual data result for chapter 5 #536

Conversation

nickyfantasy
Copy link
Contributor

No description provided.

@nickyfantasy nickyfantasy changed the title Compare predict and actual data result [High-Level-API]Compare predict and actual data result for chapter 5 Jun 6, 2018
@@ -452,15 +452,18 @@ Use create_lod_tensor(data, lod, place) API to generate LoD Tensor, where `data`
For example, data = [[10, 2, 3], [2, 3]] means that it contains two sequences of indices, of length 3 and 2, respectively.
Correspondingly, lod = [[3, 2]] contains one level of detail info, indicating that `data` consists of two sequences of length 3 and 2.

In this infer example, we try to predict rating of movie 'Hunchback of Notre Dame' from user id 1 info.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we maybe rephrase? "we try to predict rating of the movie 'Hunchback of Notre Dame' from the info of user id-1".

@@ -494,15 +494,18 @@
For example, data = [[10, 2, 3], [2, 3]] means that it contains two sequences of indices, of length 3 and 2, respectively.
Correspondingly, lod = [[3, 2]] contains one level of detail info, indicating that `data` consists of two sequences of length 3 and 2.

In this infer example, we try to predict rating of movie 'Hunchback of Notre Dame' from user id 1 info.
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above?

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.

Just wondering if we should rephrase. Otherwise looks pretty good

@nickyfantasy nickyfantasy merged commit 64dda6c into PaddlePaddle:high-level-api-branch Jun 7, 2018
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