-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Replace EagerTensor with Tensor #39376
Replace EagerTensor with Tensor #39376
Conversation
… merge_legacy_to_fluid
… merge_legacy_to_fluid
Thanks for your contribution! |
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 for PADDLE_ENFORCE
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 for set_tests_properties(test_imperative_lod_tensor_to_selected_rows PROPERTIES TIMEOUT 200) set_tests_properties(test_imperative_optimizer_v2 PROPERTIES TIMEOUT 150)
PR types
Function optimization
PR changes
Others
Describe
This PR using Tensor to replace EagerTensor to optimize eager performance in most cases.
Some important changes:
framework::Variable
)pten::SelectedRows
in Tensor and support grad accumulation ofpten::SelectedRows
andpten::DenseTensor
EagerTensor
a friend class ofpten::SelectedRows
to reconstruct it fromframework::Variable
toTensor
EagerUtils
to adjust new data structure.