[GLCC]Part-4: fix testcases for dy2st PyLayer #57633
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Aurelius84
approved these changes
Sep 25, 2023
| A nest structure of tensors in PaddlePaddle is tensor(s), or tuple of tensors, or | ||
| list of tensors. | ||
| ``forward_fn`` and ``backward_fn`` should return a nest structure of Variables. | ||
| A nest structure of Variables in PaddlePaddle is Variable(s), or tuple of Variables, or |
There was a problem hiding this comment.
这里其实不需要改为Variable的，因为我们在用户端正在「去Variable化」，用户了解到的都是Tensor，可以单独再提一个PR 吧
There was a problem hiding this comment.
好的，会在下一个 pr 进行修复~
Frida-a
pushed a commit
to Frida-a/Paddle
that referenced
this pull request
Oct 14, 2023
* add testcase load_inference_model * polish code for last pr * delete annots * fix docs of static_pylayer
jiahy0825
pushed a commit
to jiahy0825/Paddle
that referenced
this pull request
Oct 16, 2023
* add testcase load_inference_model * polish code for last pr * delete annots * fix docs of static_pylayer
danleifeng
pushed a commit
to danleifeng/Paddle
that referenced
this pull request
Nov 14, 2023
* add testcase load_inference_model * polish code for last pr * delete annots * fix docs of static_pylayer
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR types
Others
PR changes
Others
Description
Original requirements
It is expected that the PyLayer mechanism in the dynamic graph of the Paddle can interoperate with the @to_staitc of the Paddle's dynamic to static, supporting the custom layer of the PyLayer in the model to be perceived by the @to_static and correctly generating the static graph Program, and supporting the training of the dynamic to static and the exporting for inference.
So our main objective is to Enable PyLayer to Support @to_static. In this PR, we fix some testcases for dy2st
PyLayer.Related issues
【Feature Request】 PyLayer 功能支持动转静 @to_static 🚀: #54120
Related PR
[GLCC]Part-1: Add pylayer op to Support @to_static: #56108
[GLCC]Part-2: @to_static support for PyLayer: #56531
[GLCC]Part-3: Support jit.save and jit.load for pylayer op: #57066
Project Records
https://github.com/MarioLulab/GLCC2023-Paddle-Record
Main works
test_and_compare_with_new_irin the functioncheck_save_inference_modeloftest_save_inference_model.py.