mxnet icon indicating copy to clipboard operation
mxnet copied to clipboard
verified publisher icon apache

Handle deleted nodes in the Subgraph API builder

Open Kh4L opened this issue 4 years ago • 5 comments

Description

Some models may have non-consecutive node IDs, because of deleted nodes. This can lead to cases where some node ids > number of nodes, which breaks the Subgraph API builder.

This PR changes how the nodes are stored in simple_nodes to allow the partitioning of models with deleted nodes.

Kh4L avatar May 27 '21 02:05 Kh4L

Hey @Kh4L , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [unix-gpu, windows-cpu, centos-cpu, centos-gpu, website, windows-gpu, clang, sanity, miscellaneous, unix-cpu, edge]

Note: Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. All CI tests must pass before the PR can be merged.

mxnet-bot avatar May 27 '21 02:05 mxnet-bot

@mxnet-bot run ci [centos-cpu , unix-gpu]

Kh4L avatar Jun 02 '21 13:06 Kh4L

Jenkins CI successfully triggered : [unix-gpu, centos-cpu]

mxnet-bot avatar Jun 02 '21 13:06 mxnet-bot

@mxnet-bot run ci [unix-gpu]

Kh4L avatar Jun 07 '21 03:06 Kh4L

Jenkins CI successfully triggered : [unix-gpu]

mxnet-bot avatar Jun 07 '21 03:06 mxnet-bot