Dev replace py parallel conf proto to cfg #3810
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
…dev_move_RunInstruction_to_pybind
…dev_move_RunInstruction_to_pybind
…m/Oneflow-Inc/oneflow into dev_move_RunInstruction_to_pybind
…ace_py_parallel_conf_proto_to_cfg
…dev_move_RunInstruction_to_pybind Conflicts: oneflow/python/framework/c_api_util.py
…dev_replace_py_parallel_conf_proto_to_cfg
clackhan
requested review from
oneflow-ci-bot
and removed request for
oneflow-ci-bot
…dev_replace_py_parallel_conf_proto_to_cfg
clackhan
force-pushed
the
dev_replace_py_parallel_conf_proto_to_cfg
branch
from
42e0a11 to
be5d22e
Compare
…dev_replace_py_parallel_conf_proto_to_cfg
clackhan
force-pushed
the
dev_replace_py_parallel_conf_proto_to_cfg
branch
from
be5d22e to
fa8e715
Compare
liujuncheng
pushed a commit
that referenced
this pull request
Jun 3, 2021
* Replace the py instruction with CFG Instruction * move RunInstruction to pybind & refactor EagerOneflow's interface by cfg * use forward declaration * Remove unused import * fix code style * Adjust import order * replace py parallel_conf proto to cfg * fix test_cpu_only_user_op parallel_conf * move RunInstruction api to oneflow_api.vm * fix MakeMachineId2DeviceIdList * remove useless line in oneflow_internal.i * replace args str to cfg_obj in python callback * add forward declear of InstructionListProto * cancel forward declear of InstructionListProto * fix a name spelling mistake * use the CFG object in the Python Callback * use cfg in the py Callback * fix redundant conversions * fix template bug * fix template * virtual const tmplate cfg * fix template.cfg.h * update cfg * update cfg Co-authored-by: clackhan <han_binbin@163.com> Co-authored-by: oneflow-bot <69100618+oneflow-bot@users.noreply.github.com> Co-authored-by: oneflow-ci-bot <69100618+oneflow-ci-bot@users.noreply.github.com> Former-commit-id: cad594b
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.
No description provided.