Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Deprecate Travis CI #2773

Merged
merged 22 commits into from
Sep 11, 2020
Merged

Deprecate Travis CI #2773

merged 22 commits into from
Sep 11, 2020

Conversation

vinitra-zz
Copy link
Member

@vinitra-zz vinitra-zz commented May 14, 2020 •
edited
Loading

Remove Travis CI (as these options are covered in Linux CI and Mac OS CI).

Add C++ API tests to Linux CI. Add debug option to Mac OS CI. Revision: As this has already been done in #2908, I will solely focus on the travis ci deprecation.

@vinitra-zz vinitra-zz requested a review from a team as a code owner May 14, 2020 18:18
@vinitra-zz vinitra-zz changed the title Deprecate Travis CI, updates to Azure-Pipelines [WIP] Deprecate Travis CI, updates to Azure-Pipelines May 15, 2020
@delete-merged-branch delete-merged-branch bot deleted the pipeline-travis branch May 15, 2020 21:59
@jcwchen
Copy link
Member

jcwchen commented Jun 19, 2020

Hi @vinitra,
I was wondering that why we don't have

python onnx/backend/test/stat_coverage.py
backend-test-tools generate-data

in Windows-CI.yml?
In fact, I cannot run backend-test-tools generate-data on my Windows...

@vinitra-zz vinitra-zz restored the pipeline-travis branch June 22, 2020 20:48
@vinitra-zz vinitra-zz mentioned this pull request Jul 16, 2020
Merged
@vinitra-zz vinitra-zz changed the title [WIP] Deprecate Travis CI, updates to Azure-Pipelines Deprecate Travis CI, updates to Azure-Pipelines Jul 16, 2020
@vinitra-zz vinitra-zz mentioned this pull request Jul 17, 2020
Merged
@vinitra-zz vinitra-zz changed the title Deprecate Travis CI, updates to Azure-Pipelines Deprecate Travis CI Jul 17, 2020
@vinitra-zz vinitra-zz mentioned this pull request Jul 17, 2020
Merged
@vinitra-zz
Copy link
Member Author

@askhade -- ready to merge?

@askhade askhade merged commit 4065351 into onnx:master Sep 11, 2020
daquexian pushed a commit to daquexian/onnx that referenced this pull request Sep 19, 2020
@vinitra @daquexian
Deprecate Travis CI (onnx#2773)
8dd2a4e 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file

Signed-off-by: daquexian <daquexian566@gmail.com>
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Sep 22, 2020
@vinitra @jcwchen
Deprecate Travis CI (onnx#2773)
8396439 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Sep 22, 2020
@vinitra @jcwchen
Deprecate Travis CI (onnx#2773)
3a80b41 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Sep 23, 2020
@vinitra @jcwchen
Deprecate Travis CI (onnx#2773)
c839fd8 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Sep 23, 2020
@vinitra @jcwchen
Deprecate Travis CI (onnx#2773)
e454928 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file
gramalingam added a commit that referenced this pull request Sep 23, 2020
@jcwchen @gramalingam
@linkerzhang @askhade @vinitra @shinh @ashbhandare @daquexian @wschin
Correct wrong description for multidirectional broadcastable Pow (#2999)
f8dec35 
* remove wrong description for pow

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Add size check to make_tensor (#2987)

Co-authored-by: Ke Zhang <linkerzhang@yeah.net>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Fix float16 data convert issue in numpy_helper.to_array (#3002)

* handle f16 case for to_array

* fix flake8

* nit: comment

Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Deprecate Travis CI (#2773)

* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Fix shape inference of scalar ConstantOfShape (#3005)

When input shape is (0), we do not add any dim to inferred
shape but we should initialize tensor_type.shape by calling
mutable_shape().

Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* fix shape inference for loop (#3014)

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Fix ConvTranspose: enhance attribute check (#3000)

* add check for using auto_pad and pads simultaneously

* fix description for auto_pads == SAME_UPPER

* update docs for operator

* fix the old one as well

* add a test

* Revert "fix description for auto_pads == SAME_UPPER"

This reverts commit e75e287.

* Revert "update docs for operator"

This reverts commit 70952c0.

* Revert "fix the old one as well"

This reverts commit 8a0482d.

Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Schema change to support dynamic shapes in ORT (#2955)

* Changes to schema and python tests

* Modify test

* Remove attribute that is input also

* Changes to optimizers, adapters and tests

* Run flake8

* undo unrequired comit files, fix formatting, review changes

* Fix ci test, cleanup

* Fix narrowing conversion error

* add missed test model

Co-authored-by: G. Ramalingam <grama@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* fix loop shape inference for ver 11 (#3023)

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Update softmax family ops behavior to align with other frameworks (fix #2289) (#2879)

* Update softmax family ops behavior to align with other frameworks

* Update logsoftmax, hardmax tests, regenerate docs and test data

* fix wrong input name in function

* regenerate test data

* fix flake8 error

* regenerate docs

* regenerate docs

* add missing type annotation for hardmax

* add the math for softmax family operators

* remove the 'description' field in docs as it is covered by the math

* fix wrong format in axis attr

* replace name with description

* restore the name field for axis attr

* regenerate docs

* regenerate docs

* add the missing name

* regenerate docs

* update reducesum to align with master

* regenerate tests

Co-authored-by: Wei-Sheng Chin <wschin@outlook.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

Co-authored-by: G. Ramalingam <grama@microsoft.com>
Co-authored-by: Ke Zhang <linkerzhang@yeah.net>
Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Co-authored-by: Vinitra Swamy <vinitras@gmail.com>
Co-authored-by: Shinichiro Hamaji <shinichiro.hamaji@gmail.com>
Co-authored-by: ashbhandare <ash.bhandare@gmail.com>
Co-authored-by: daquexian <daquexian566@gmail.com>
Co-authored-by: Wei-Sheng Chin <wschin@outlook.com>
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Oct 8, 2020
@vinitra @jcwchen
Deprecate Travis CI (onnx#2773)
b359d84 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file
jcwchen added a commit to jcwchen/onnx that referenced this pull request Oct 8, 2020
@jcwchen @gramalingam
@linkerzhang @askhade @vinitra @shinh @ashbhandare @daquexian @wschin
Correct wrong description for multidirectional broadcastable Pow (onn…
d0b9be8 
…x#2999)

* remove wrong description for pow

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Add size check to make_tensor (onnx#2987)

Co-authored-by: Ke Zhang <linkerzhang@yeah.net>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Fix float16 data convert issue in numpy_helper.to_array (onnx#3002)

* handle f16 case for to_array

* fix flake8

* nit: comment

Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Deprecate Travis CI (onnx#2773)

* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Fix shape inference of scalar ConstantOfShape (onnx#3005)

When input shape is (0), we do not add any dim to inferred
shape but we should initialize tensor_type.shape by calling
mutable_shape().

Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* fix shape inference for loop (onnx#3014)

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Fix ConvTranspose: enhance attribute check (onnx#3000)

* add check for using auto_pad and pads simultaneously

* fix description for auto_pads == SAME_UPPER

* update docs for operator

* fix the old one as well

* add a test

* Revert "fix description for auto_pads == SAME_UPPER"

This reverts commit e75e287.

* Revert "update docs for operator"

This reverts commit 70952c0.

* Revert "fix the old one as well"

This reverts commit 8a0482d.

Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Schema change to support dynamic shapes in ORT (onnx#2955)

* Changes to schema and python tests

* Modify test

* Remove attribute that is input also

* Changes to optimizers, adapters and tests

* Run flake8

* undo unrequired comit files, fix formatting, review changes

* Fix ci test, cleanup

* Fix narrowing conversion error

* add missed test model

Co-authored-by: G. Ramalingam <grama@microsoft.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* fix loop shape inference for ver 11 (onnx#3023)

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

* Update softmax family ops behavior to align with other frameworks (fix onnx#2289) (onnx#2879)

* Update softmax family ops behavior to align with other frameworks

* Update logsoftmax, hardmax tests, regenerate docs and test data

* fix wrong input name in function

* regenerate test data

* fix flake8 error

* regenerate docs

* regenerate docs

* add missing type annotation for hardmax

* add the math for softmax family operators

* remove the 'description' field in docs as it is covered by the math

* fix wrong format in axis attr

* replace name with description

* restore the name field for axis attr

* regenerate docs

* regenerate docs

* add the missing name

* regenerate docs

* update reducesum to align with master

* regenerate tests

Co-authored-by: Wei-Sheng Chin <wschin@outlook.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>

Co-authored-by: G. Ramalingam <grama@microsoft.com>
Co-authored-by: Ke Zhang <linkerzhang@yeah.net>
Co-authored-by: Ashwini Khade <askhade@microsoft.com>
Co-authored-by: Vinitra Swamy <vinitras@gmail.com>
Co-authored-by: Shinichiro Hamaji <shinichiro.hamaji@gmail.com>
Co-authored-by: ashbhandare <ash.bhandare@gmail.com>
Co-authored-by: daquexian <daquexian566@gmail.com>
Co-authored-by: Wei-Sheng Chin <wschin@outlook.com>
jcwchen pushed a commit to jcwchen/onnx that referenced this pull request Oct 8, 2020
@vinitra @jcwchen
Deprecate Travis CI (onnx#2773)
2f027d2 
* adding gtests for c++ api test

* deprecate travis

* adding the build badges for the new pipelines in Azure, deprecating travis build badge, renaming circleCI badge

* updating badge label

* removing - in badge names

* c++ api changes for linux

* update environment variables

* update env variables, setup tools call

* Update Linux-CI.yml for Azure Pipelines

* revert changes to Linux and Mac CIs

* delete last travis file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jcwchen jcwchen jcwchen left review comments

@askhade askhade askhade approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vinitra-zz @jcwchen @askhade @vinitra