Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Successive unpack #2768

Merged
merged 5 commits into from
Aug 12, 2020
Merged

Conversation

zheng-ningxin
Copy link
Contributor

Support the multiple successive tuple unpack operations.

Ningxin added 3 commits August 6, 2020 03:00
Signed-off-by: Ningxin <Ningxin.Zheng@microsoft.com>
Signed-off-by: Ningxin <Ningxin.Zheng@microsoft.com>
Signed-off-by: Ningxin <Ningxin.Zheng@microsoft.com>
@ultmaster
Copy link
Contributor

Can you add unittests for this fix?

Ningxin added 2 commits August 10, 2020 05:56
Signed-off-by: Ningxin <Ningxin.Zheng@microsoft.com>
Signed-off-by: Ningxin <Ningxin.Zheng@microsoft.com>
if _debug_output in self.input_to_node:
for following_node in self.input_to_node[_debug_output]:
_tmp_index = following_node.inputs.index(_debug_output)
following_node.inputs[_tmp_index] = _debug_input
Copy link
Contributor

Choose a reason for hiding this comment

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

much clear for this logic

@ultmaster ultmaster mentioned this pull request Aug 12, 2020
66 tasks
@chicm-ms chicm-ms merged commit 5d2a59f into microsoft:master Aug 12, 2020
LovPe pushed a commit to LovPe/nni that referenced this pull request Aug 17, 2020
(cherry picked from commit 5d2a59f)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants