Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

[UT] mv compression ut to full test #4936

Merged
merged 9 commits into from
Jun 20, 2022
Merged

Conversation

J-shang
Copy link
Contributor

@J-shang J-shang commented Jun 14, 2022

Description

Test Options

  • fast test
  • full test - HPO
  • full test - NAS
  • full test - compression

Checklist

  • test case
  • doc

How to test

@@ -13,7 +13,7 @@ resources:
endpoint: github-filter-connection

variables:
filter.modified.globs: 'examples/model_compress/**,nni/algorithms/compression/**,nni/compression/**'
filter.modified.globs: 'examples/model_compress/**,nni/algorithms/compression/**,nni/compression/**,pipelines/full-test-compression.yml,test/algo/compression'
Copy link
Contributor

Choose a reason for hiding this comment

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

** needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nice catch, fixed

@liuzhe-lz
Copy link
Contributor

The pipeline works well on master. Seems the error is caused by this PR.

@ultmaster
Copy link
Contributor

The error is caused by missing __init__.py. I've already introduced one in #4933. Maybe you can wait for a while.

@J-shang J-shang closed this Jun 17, 2022
@J-shang J-shang reopened this Jun 17, 2022
@@ -13,7 +13,7 @@ resources:
endpoint: github-filter-connection

variables:
filter.modified.globs: 'examples/model_compress/**,nni/algorithms/compression/**,nni/compression/**'
filter.modified.globs: 'examples/model_compress/**,nni/algorithms/compression/**,nni/compression/**,pipelines/full-test-compression.yml,test/algo/compression/**'
Copy link
Contributor

Choose a reason for hiding this comment

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

This is really long. Can we use line break here?

@J-shang J-shang merged commit 45af1d6 into microsoft:master Jun 20, 2022
@J-shang J-shang deleted the comp-ut branch June 30, 2022 01:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants