Skip to content

Conversation

@xiaoyu-work
Copy link
Collaborator

With newer NNCF, it is necessary to specify group size fallback in advanced compression parameters, which is reflected in a few unit tests that fail due to InvalidGroupSizeError.
Also updated the Intel® NNCF Compression pass in Olive OpenVINO passes to accurately handle advanced parameters in both enum and string forms

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

Describe your changes

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

With newer NNCF, it is necessary to specify group size fallback in
advanced compression parameters, which is reflected in a few unit tests
that fail due to InvalidGroupSizeError.
Also updated the Intel® NNCF Compression pass in Olive OpenVINO passes
to accurately handle advanced parameters in both enum and string forms

- [ ] Add unit tests for this change.
- [x] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [x] Lint and apply fixes to your code by running `lintrunner -a`
- [ ] Is this a user-facing change? If yes, give a description of this
change to be included in the release notes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants