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.

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

[Minor] Downgrading mypy from latest (1.11) to 1.10 #31

Merged
merged 10 commits into from
Aug 2, 2024

Conversation

mtopalovicTT
Copy link
Contributor

Latest mypy depends on typing-extension >= 4.6 which conflicts with tensorflow 2.13 version which requires < 4.6. Downgrading mypy to 1.10 solves this issue.

Tested:

  • Builds tt-forge OK
  • Run simple test OK
  • Run make_stubs OK - no conflicts and no diff in generated stubs

tt-mjudge and others added 9 commits August 2, 2024 10:10
(cherry picked from commit d8cb16db58c63d101bda5137b7a83ca6954d761f)
(cherry picked from commit 116dd3b147fd6f31968febc2d24c4c92c0b2a533)
(cherry picked from commit 48e64b159e17114ec083102170f39f58a524b9e0)
(cherry picked from commit 179d7adc3e822056f0a0cc242356963b0cd3ce1c)
…_models and nightly test

and configuration file

(cherry picked from commit 26fb354ffb75f37e8394f79ff2566e91287d14a5)
(cherry picked from commit 88b29efb09cdf658b9a05ce977f99285bd242205)
(cherry picked from commit 1283b4bd3d5e7ed53f75d20b3b165bafe004beb6)
Latest `mypy` depends on `typing-extension` >= 4.6 which conflicts with
tensorflow which requires < 4.6. Downgrading `1.10` solves this issue.

Tested:
* Builds tt-forge OK
* Run simple test OK
* Run make_stups OK - no conflicts and no diff in generated stubs
@mtopalovicTT mtopalovicTT force-pushed the milant/downgrade_mypy_version branch from 18b16b0 to a782031 Compare August 2, 2024 10:17
@mtopalovicTT mtopalovicTT merged commit 2ea777f into main Aug 2, 2024
1 check failed
@pilkicTT pilkicTT deleted the milant/downgrade_mypy_version branch August 5, 2024 12:56
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.

9 participants