-
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
[new-exec] Refine standalone executor #37278
[new-exec] Refine standalone executor #37278
Conversation
Thanks for your contribution! |
d897ab8
to
208ad0d
Compare
python/paddle/fluid/executor.py
Outdated
fetch_list(list): This parameter represents the Tensors that need to be returned | ||
after the model runs. The default is None. | ||
return_numpy(bool): This parameter indicates whether convert the fetched Tensors | ||
(the Tensor specified in the fetch list) to numpy.ndarray. if it is False, | ||
the type of the return value is a list of :code:`LoDTensor`. The default is True. | ||
""" | ||
feed = self._update_feed(feed) | ||
# feed = self._update_feed(feed) |
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.
注释的代码后续可以考虑remove掉
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.
done, thx
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
6e0241f
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
PR types
Others
PR changes
Others
Describe
Refine standalone executor
feed
ops in python sides to be consistent with old executor and support more feed type