Title: Incorrect "redundant cast" for linked TypeVar `T` and `type[T]` variables · Issue #17045 · python/mypy · GitHub
Open Graph Title: Incorrect "redundant cast" for linked TypeVar `T` and `type[T]` variables · Issue #17045 · python/mypy
X Title: Incorrect "redundant cast" for linked TypeVar `T` and `type[T]` variables · Issue #17045 · python/mypy
Description: Bug Report Given a TypeVar T with multiple, concrete constraints, if you write a generic function that takes an argument of type T, another argument of type type[T], and return a value of type T, mypy does not seem to have consistent beh...
Open Graph Description: Bug Report Given a TypeVar T with multiple, concrete constraints, if you write a generic function that takes an argument of type T, another argument of type type[T], and return a value of type T, m...
X Description: Bug Report Given a TypeVar T with multiple, concrete constraints, if you write a generic function that takes an argument of type T, another argument of type type[T], and return a value of type T, m...
Opengraph URL: https://github.com/python/mypy/issues/17045
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Incorrect \"redundant cast\" for linked TypeVar `T` and `type[T]` variables","articleBody":"\u003c!--\r\nIf you're not sure whether what you're experiencing is a mypy bug, please see the \"Question and Help\" form instead.\r\nPlease consider:\r\n\r\n- checking our common issues page: https://mypy.readthedocs.io/en/stable/common_issues.html\r\n- searching our issue tracker: https://github.com/python/mypy/issues to see if it's already been reported\r\n- asking on gitter chat: https://gitter.im/python/typing\r\n--\u003e\r\n\r\n**Bug Report**\r\n\r\n\u003c!--\r\nIf you're reporting a problem with a specific library function, the typeshed tracker is better suited for this report: https://github.com/python/typeshed/issues\r\n\r\nIf the project you encountered the issue in is open source, please provide a link to the project.\r\n--\u003e\r\n\r\nGiven a TypeVar `T` with multiple, concrete constraints, if you write a generic function that takes an argument of type `T`, another argument of type `type[T]`, and return a value of type `T`, mypy does not seem to have consistent behavior when it comes to handling the return type of individual expressions. Attempting to return a value without casting hits an `Incompatible return value type` error, while attempting to cast causes a `Redundant cast` error.\r\n\r\n**To Reproduce**\r\n\r\n```python\r\nfrom decimal import Decimal\r\nfrom typing import TypeVar, cast\r\n\r\nT = TypeVar(\"T\", Decimal, float)\r\n\r\n\r\ndef foo(x: T, x_type: type[T]) -\u003e T:\r\n if x_type == float:\r\n return cast(\"T\", 1.0)\r\n if x_type == Decimal:\r\n return cast(\"T\", Decimal(\"2.0\"))\r\n raise ValueError(\"unreachable\")\r\n\r\n\r\ndef foo2(x: T, x_type: type[T]) -\u003e T:\r\n if x_type == float:\r\n return 1.0\r\n if x_type == Decimal:\r\n return Decimal(\"2.0\")\r\n raise ValueError(\"unreachable\")\r\n```\r\n\r\nhttps://mypy-play.net/?mypy=1.9.0\u0026python=3.12\u0026flags=strict\u0026gist=4a0bc37034dcbcd5a18fabebeb5cbff5\r\n\r\n**Expected Behavior**\r\n\r\n\u003c!--\r\nHow did you expect mypy to behave? It’s fine if you’re not sure your understanding is correct.\r\nWrite down what you thought would happen. If you expected no errors, delete this section.\r\n--\u003e\r\nI expected either one of `foo()` or `foo2()` to be accepted by mypy. I don't necessarily expect mypy to be smart enough to infer the the linkage between narrowing the type of `x_type` and relating it to the type of `x`, but I at least expect to be able to explicitly cast the return expression without hitting a `Redundant cast` error.\r\n\r\n**Actual Behavior**\r\n\r\n\u003c!-- What went wrong? Paste mypy's output. --\u003e\r\n```\r\nmain.py:9: error: Redundant cast to \"float\" [redundant-cast]\r\nmain.py:11: error: Redundant cast to \"Decimal\" [redundant-cast]\r\nmain.py:17: error: Incompatible return value type (got \"float\", expected \"Decimal\") [return-value]\r\nmain.py:19: error: Incompatible return value type (got \"Decimal\", expected \"float\") [return-value]\r\nFound 4 errors in 1 file (checked 1 source file)\r\n```\r\n\r\n**Your Environment**\r\n\r\n\u003c!-- Include as many relevant details about the environment you experienced the bug in --\u003e\r\n\r\n- Mypy version used: 1.7.1 (but the mypy Playground I linked to above was 1.9.0)\r\n- Mypy command-line flags:\r\n- Mypy configuration options from `mypy.ini` (and other config files):\r\n ```toml\r\n warn_unused_ignores = true\r\n warn_redundant_casts = true\r\n warn_unused_configs = true\r\n warn_unreachable = true\r\n warn_return_any = true\r\n strict = true\r\n disallow_untyped_decorators = true\r\n disallow_any_generics = false\r\n implicit_reexport = false\r\n show_error_codes = true\r\n ```\r\n- Python version used: 3.11.6 (mypy Playground was 3.12)\r\n\r\n\u003c!-- You can freely edit this text, please remove all the lines you believe are unnecessary. --\u003e\r\n","author":{"url":"https://github.com/richardxia","@type":"Person","name":"richardxia"},"datePublished":"2024-03-18T06:04:21.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/17045/mypy/issues/17045"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:907f8189-3c43-e3bd-c9b5-34247bffdf93 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | D48E:16DF74:5F6918:89BD29:6A58BB09 |
| html-safe-nonce | d988f6f7f30dea5ddac933d4cbca97eeaed146407424ee8bc0ca63d80d285711 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJENDhFOjE2REY3NDo1RjY5MTg6ODlCRDI5OjZBNThCQjA5IiwidmlzaXRvcl9pZCI6IjMxMDkwNTU5MDAyNjg2NzM4MDEiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 54252d6989337fdcf74e10bcd8b3511873e15e202fb39cbea087bdbedc70f9c4 |
| hovercard-subject-tag | issue:2191307785 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/mypy/17045/issue_layout |
| twitter:image | https://opengraph.githubassets.com/c8fe5a7d25d44eb44905ae16dfb1560e73efc6ed2be59399fbeb00481614480c/python/mypy/issues/17045 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/c8fe5a7d25d44eb44905ae16dfb1560e73efc6ed2be59399fbeb00481614480c/python/mypy/issues/17045 |
| og:image:alt | Bug Report Given a TypeVar T with multiple, concrete constraints, if you write a generic function that takes an argument of type T, another argument of type type[T], and return a value of type T, m... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | richardxia |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f2a0c7865178af3d91dd9f13b0cdfc3c73a2529c873d2780bb4c01271a57ec6 |
| turbo-cache-control | no-preview |
| go-import | github.com/python/mypy git https://github.com/python/mypy.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 7053637 |
| octolytics-dimension-repository_nwo | python/mypy |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 7053637 |
| octolytics-dimension-repository_network_root_nwo | python/mypy |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 8aae7b8d6caacacf5c66eaeb2702d8dc88d85b4a |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width