-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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.
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
repacle embedding in fluid with 2.0 version #55757
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
c4b77aa
to
7cfa80b
Compare
input=x1, | ||
size=[10000, 10], | ||
size=[10000, 13], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里改动size的原因是?
7cfa80b
to
fbb9415
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* replace embedding * replace sparse_embedding * fix some bugs * del embedding * repalce layers.embedding * fix type error
PR types
Others
PR changes
APIs
Description
To clean the code in paddle.fluid, the useless codes should be removed and code still be used should move to a new position.
For function paddle.fluid.layers.nn.embedding, since it is not used in version 2.x, should be removed.