Remove pydantic<2.0 pin in requirements.txt#1398
Remove pydantic<2.0 pin in requirements.txt#1398lior-dikstein merged 10 commits intoSonySemiconductorSolutions:mainfrom
pydantic<2.0 pin in requirements.txt#1398Conversation
Request to unpin this dependency as this is causing installation conflicts in the Ultralytics package when used with other export dependencies that require more recent versions of Pydantic. I think the main changes may be needed in `model_compression_toolkit` here:
|
Hi @glenn-jocher , So, I'm putting this PR on hold, and we will continue to update it with all the necessary changes once all verifications are done. |
|
Hi, |
|
@ofirgo @lior-dikstein great guys, understood, thank you for the updates! |
| tpc_minor_version: Optional[int] | ||
| tpc_patch_version: Optional[int] | ||
| tpc_platform_type: Optional[str] | ||
| operator_set: Optional[Tuple[OperatorsSet, ...]] = None |
There was a problem hiding this comment.
Are these default values required?
we prefer no to set default values to TPC attributes
There was a problem hiding this comment.
Ok I think there is no risk in adding this default value in this case
...it_tests/core/quantization/node_quantization_config/test_node_weights_quantization_config.py
Show resolved
Hide resolved
660ba89
into
SonySemiconductorSolutions:main
Request to unpin this dependency as this is causing installation conflicts in the Ultralytics package when used with other export dependencies that require more recent versions of Pydantic.
I think the main changes may be needed in
model_compression_toolkithere as this error shows when running IMX export usingmodel_compression_toolkitwith the latestpydantic:Pull Request Description:
Checklist before requesting a review: