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

Upsample #115

Merged
merged 14 commits into from
Jul 5, 2023
Merged

Upsample #115

merged 14 commits into from
Jul 5, 2023

Conversation

LokeZhou
Copy link
Contributor

@LokeZhou LokeZhou commented Jun 21, 2023

PR Docs

这些api均有映射文档

PR APIs

torch.nn.Upsample
torch.utils.data.SequentialSampler
torch.utils.cpp_extension.CUDAExtension
torch.utils.cpp_extension.CppExtension
torch.Tensor.is_sparse

@paddle-bot
Copy link

paddle-bot bot commented Jun 21, 2023

Thanks for your contribution!

@zhwesky2010
Copy link
Collaborator

PR Docs

torch.nn.Upsample torch.utils.data.SequentialSampler torch.utils.cpp_extension.CUDAExtension torch.utils.cpp_extension.CppExtension torch.Tensor.is_sparse

PR APIs

torch.nn.Upsample
torch.utils.data.SequentialSampler
torch.utils.cpp_extension.CUDAExtension
torch.utils.cpp_extension.CppExtension
torch.Tensor.is_sparse

PR描述直接放docs的PR链接,已经有了的就写 xxx已有映射文档

@@ -3645,3 +3645,20 @@ class SizeAverageMatcher(BaseMatcher):
def generate_code(self, kwargs):
process_reduce_and_size_average(kwargs)
return GenericMatcher.generate_code(self, kwargs)


class UtilsCppExtensionMatcher(BaseMatcher):
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个直接用genericamatcher,如果你要删掉某个参数,可设置kwargs_change: name: ""

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

obj.run(pytorch_code, ["result"])


test_case_1()
Copy link
Collaborator

@zhwesky2010 zhwesky2010 Jun 26, 2023

Choose a reason for hiding this comment

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

这个需要删掉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -3539,6 +3539,13 @@ def generate_code(self, kwargs):
return GenericMatcher.generate_code(self, kwargs)


class TensorIsSpareMatcher(BaseMatcher):
Copy link
Collaborator

Choose a reason for hiding this comment

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

可以起个公用一些的名字,这样其他人也可以复用了

就叫Attribute2Func吧

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

result = m(input)
"""
)
obj.run(pytorch_code, unsupport=True, reason="paddle unsupport")
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个原因要写具体一点

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Collaborator

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhwesky2010 zhwesky2010 merged commit fafdd56 into PaddlePaddle:master Jul 5, 2023
@LokeZhou LokeZhou deleted the upsample branch July 6, 2023 09:46
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.

2 participants