Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix use-before-assignment in convert_dot #13511

Merged
merged 1 commit into from
Dec 8, 2018

Conversation

zhipeng-jia
Copy link
Contributor

Description

Fix use-before-assignment bug of input_node_a and input_node_b when trans_a/trans_b is False.

@vandanavk
Copy link
Contributor

@mxnet-label-bot add [ONNX, pr-awaiting-review]

@Roshrini for review. She had a fix for this in PR #12946

@marcoabreu marcoabreu added ONNX pr-awaiting-review PR is waiting for code review labels Dec 3, 2018
@Roshrini
Copy link
Member

Roshrini commented Dec 7, 2018

@zhipeng-jia Thanks for fixing this.

@nswamy nswamy merged commit 7d74452 into apache:master Dec 8, 2018
zhaoyao73 pushed a commit to zhaoyao73/incubator-mxnet that referenced this pull request Dec 13, 2018
zhaoyao73 added a commit to zhaoyao73/incubator-mxnet that referenced this pull request Dec 13, 2018
* upstream/master: (54 commits)
  Add notes about debug with libstdc++ symbols (apache#13533)
  add cpp example inception to nightly test (apache#13534)
  Fix exception handling api doc (apache#13519)
  fix link for gluon model zoo (apache#13583)
  ONNX import/export: Size (apache#13112)
  Update MXNetTutorialTemplate.ipynb (apache#13568)
  fix the situation where idx didn't align with rec (apache#13550)
  Fix use-before-assignment in convert_dot (apache#13511)
  License update  (apache#13565)
  Update version to v1.5.0 including clojure package (apache#13566)
  Fix flaky test test_random:test_randint_generator (apache#13498)
  Add workspace cleaning after job finished (apache#13490)
  Adding test for softmaxoutput (apache#13116)
  apache#13441 [Clojure] Add Spec Validations for the Random namespace (apache#13523)
  Revert "Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file" (apache#13558)
  Chi_square_check for discrete distribution fix (apache#13543)
  Updated docs for randint operator (apache#13541)
  Simplifications and some fun stuff for the MNIST Gluon tutorial (apache#13094)
  Fix apache#13521 (apache#13537)
  Add a retry to qemu_provision (apache#13551)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ONNX pr-awaiting-review PR is waiting for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants