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

Cyphers/m28 #4179

Merged
merged 13 commits into from
Jan 14, 2020
Merged

Cyphers/m28 #4179

merged 13 commits into from
Jan 14, 2020

Conversation

diyessi
Copy link
Contributor

@diyessi diyessi commented Jan 14, 2020

Re-sync 0.28 with master.

Mateusz Bencer and others added 13 commits January 10, 2020 17:52
* Added round op

* Add CPU support, unit tests

* Disable UT for PlaidML

* Update year

Co-authored-by: Scott Cyphers <diyessi@users.noreply.github.com>
* Removed LogSoftmax FusedOp, changed onnx to produce v1

* Code review remakrs introduced

* fix after merge from master

Co-authored-by: Scott Cyphers <diyessi@users.noreply.github.com>
* Move GCPU to runtime/gcpu so that unit test manifest works

* style

* Add tests to manifest

* Fix CPU only check

* Update GCPU dependency

Co-authored-by: Scott Cyphers <diyessi@users.noreply.github.com>
#4164)

* Restore constant folding for DynReshape until users are converted to v1 Reshape

* Disbale test when no serialization

Co-authored-by: baojun <32073718+baojun-nervana@users.noreply.github.com>
* Update sitemap to not use a page title

* Document Release Notes for 0.28.0; update section on Distributed Training

Co-authored-by: Scott Cyphers <diyessi@users.noreply.github.com>
Co-authored-by: Scott Cyphers <diyessi@users.noreply.github.com>
* AliasMap WIP

* Added liveness info

* WIP

* WIP: Tests

* WIP: LIT tests

* Added knobs for mem optimization pass. More LIT tests

* Revert affine_lowerer change

* More elaborate comment

* Minor fixes

* style-apply

* Rename liveness

* Switch to Analysis framework

* Fix optimization conditions

* Remove LIT test

* style

* Switch to equivalence relationship impl of non-alias relationship

* refined comment

* Switch non-alias to equivalence relationship

* Fix bad merge

* Adding tests. WIP

* Added buffer size tracking and unit-tests

* Added LIT and unit-tests

* Turn optimization ON

* style

* fix unit-tests

* Fix useCount

* Fix copyright and typo

* Refine few comments, remove new lines

* style fix

Co-authored-by: Scott Cyphers <diyessi@users.noreply.github.com>
Co-authored-by: Sang Ik Lee <sang.ik.lee@intel.com>
* [PY] Add flag to compile python API in debug mode

* Move debug and release flags to helper function

* Add missing Docstring

Co-authored-by: Sang Ik Lee <sang.ik.lee@intel.com>
* Checking if provenance_tags key exists

* Add provenance tag prototype

* Format provenance tag

* Display provenance tag

* Clean debug printing

* Add const to variables

* Separate method for add provenance tags

* Return NodeVector reference

* Return const NodeVector

* Moved add_provenance_tags function to commons

* Style apply

* Simple model for tests

* Provenance tag test

* Expect substring instead of  equal

* Add provenance tags to intermediate nodes recursively

* One tag per node

* Add traverse node args instead of recursion

* Return NodeVector instead of set of pointers

* Use treverse_nodes and lambda function

* Remove unused helper functions

* Remove is_constant() condition

* Update test model prototxt

* Update test substring

* Use node name and output names to build provenance tags in onnx importer

* Unit tests for onnx_importer provenance tags

* Missing <numeric> include

* Add provenance tags to constants buit from ONNX initializers

* Add provenance tags to Constants and Parameters created out of ONNX inputs and initializers

* More strict assertions in onnx provenance tests

* Unit test for onnx importer Parameter nodes tagging

* Helper function for the onnx provenance tests

* Some docs

* Obsolete comment removal

* Separate file for onnx provenance tags unit tests

* Code formatting

* Move the inputs tagging to the Graph class

* Tagging moved to the Graph class entirely

* Missing include and extra helper variable

* Unit tests helper documentation

* Change the UT helper to lowercase

Co-authored-by: Katarzyna Mitrus <katarzyna.mitrus@intel.com>
* [ONNX] Remove unnecesary Mean in opset 8

* Use opset1 instead of v1

* Fix merge artifact

* Use opset1 Divide instead of opset0

Co-authored-by: Robert Kimball <robert.kimball@intel.com>
@diyessi diyessi added the DO NOT SQUASH Do not squash this when merging. label Jan 14, 2020
Copy link

@mbrookhart mbrookhart left a comment

Choose a reason for hiding this comment

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

Don't Squash!

@diyessi diyessi changed the base branch from master to r0.28 January 14, 2020 20:48
@mbrookhart mbrookhart added the Fully Reviewed PR is approved by all reviewers. label Jan 14, 2020
@diyessi diyessi merged commit d2cd873 into r0.28 Jan 14, 2020
@diyessi diyessi deleted the cyphers/m28 branch January 14, 2020 21:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DO NOT SQUASH Do not squash this when merging. Fully Reviewed PR is approved by all reviewers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants