Skip to content

Fix some backend tests #2335

Merged
wschin merged 5 commits intoonnx:masterfrom
hariharans29:FixSomeNodetests
Sep 19, 2019
Merged

Fix some backend tests #2335
wschin merged 5 commits intoonnx:masterfrom
hariharans29:FixSomeNodetests

Conversation

@hariharans29
Copy link
Copy Markdown
Contributor

@hariharans29 hariharans29 commented Sep 18, 2019

  1. Minor self-explanatory bug in a Gather test case
  2. incorrect input type for a Gemm test case
  3. Update the description of Softmax family to indicate the inputs need not be 2D
  4. add shape inference test cases for the Softmax family

@hariharans29 hariharans29 requested review from a team as code owners September 18, 2019 23:16
Comment thread onnx/defs/math/defs.cc
@wschin wschin added the bug label Sep 18, 2019
@wschin wschin added this to the 1.6 milestone Sep 18, 2019
Comment thread onnx/defs/math/defs.cc
@wschin wschin merged commit bd35e62 into onnx:master Sep 19, 2019
kevinch-nv pushed a commit that referenced this pull request Sep 23, 2019
* Gemm optional bias (#2330)

* Made the 'C' input of Gemm (the bias term) optional.

If missing it defaults to 0.

Also added a test case for no bias.

Updated the Gemm op to version 11.

* Fixed a typo!

* Small tweaks to the Gemm docs.

* Added a shape inference test for Gemm with no bias

* Tweaked coding style slightly by adding braces to single line scopes.

* Fix some backend tests  (#2335)

* Fix some node tests

* PR comments and docs

* Update Changelog.md

* Update gen_doc script to validate proto3 files (#2122)

* Update gen_doc script to validate proto3 files

* Update CMakeLists.txt

* Update pybind (#2340)

* Fix node test case model for Gemm scalar bias case (#2342)

* Fix some node tests

* PR comments and docs

* Update Changelog.md

* Fix gemm scalar node test

* Clarify behavior in ConvTranspose (#2343)

* Fix the wrong behavior in ConvTranspose

* Address comments
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Sep 23, 2020
* Fix some node tests

* PR comments and docs

* Update Changelog.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants