[NNCF] Update test references for NNCF 3.1.0#1678
[NNCF] Update test references for NNCF 3.1.0#1678andrey-churkin wants to merge 2 commits intohuggingface:mainfrom
Conversation
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
andrey-churkin
force-pushed
the
ac/nncf_3.1.0
branch
from
5d0bddd to
f0c25b9
Compare
andrey-churkin
force-pushed
the
ac/nncf_3.1.0
branch
from
f0c25b9 to
74940a0
Compare
| "opt125m", | ||
| "mxfp4", | ||
| {"model": {"int8": 4, "f4e2m1": 72, "f8e8m0": 72}}, | ||
| {"model": {"int8": 0, "f4e2m1": 72, "f8e4m3": 2, "f8e8m0": 74}}, |
There was a problem hiding this comment.
why was it changed and now there is no int8 coeffs?
There was a problem hiding this comment.
We recently merged a new feature into NNCF. Please see the openvinotoolkit/nncf#3886. The references have been updated to reflect the current state.
| "transformers>=4.45,<4.58", | ||
| "setuptools", | ||
| "nncf>=2.19.0", | ||
| "nncf>=3.1.0", |
There was a problem hiding this comment.
why is it recommended to install 3.1.0 by default? Do we have any model requires it? Please recommend.
There was a problem hiding this comment.
I bumped the minimum nncf version to 3.1.0 because there is a test that passes only with nncf ≥ 3.1.0, and the default behavior of the --backup-precision parameter was extended in that release. Please see the updated docs.
rkazants
left a comment
rkazants
left a comment
There was a problem hiding this comment.
let us move to the latest OV. After that we will merge this PR
rkazants
left a comment
rkazants
left a comment
There was a problem hiding this comment.
@andrey-churkin, please rebase your PR. We recently merged PR to move to the latest OpenVINO
3 participants
What does this PR do?