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.

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

[Unify Tensors PR #6] Removed interfaces & members from lod_tensor,test=allcases #38811

Merged

Conversation

jim19930609
Copy link
Contributor

@jim19930609 jim19930609 commented Jan 8, 2022

PR types

Others

PR changes

Others

Describe

Previous works on constructing new tensor library "pten" introduced another Tensor type namely "DenseTensor", which temporarily co-exists with framework::Tensor and LoDTensor. In this series of PRs, we attempt to replace legacy "framework::Tensor and LoDTensor" by the newly implemented "DenseTensor", so as to facilitate the progress of "pten" joining original framework.

In this PR, we moved nearly all the members and interfaces from framework::LoDTensor to pten::DenseTensor. framework::LoDTensor essentially inherits from pten::DenseTensor through framework::Tensor, thus all the original access to framework::LoDTensor interfaces will be routed to accessing from the corresponding interfaces on pten::DenseTensor.

image

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jan 8, 2022

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@jim19930609 jim19930609 merged commit 953638e into PaddlePaddle:develop Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants