Mitigate coremltool issue with python 3.13

[metascroy]

No description provided.

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/11317

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

• Experiencing "429: Too Many Requests" on downloading actions

❌ 1 New Failure, 1 Pending

As of commit c1a4902 with merge base 5ef38d3 ():

NEW FAILURE - The following job has failed:

• trunk / test-llama-torchao-lowbit / macos-job (gh)
  RuntimeError: Command bash /Users/ec2-user/runner/_work/_temp/exec_script failed with exit code 134

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[github-actions]

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

[swolchok]

should we link to apple/coremltools#2487 so people can easily tell when this goes stale?

[jathu]

What if we just hard enforce 3.12 as the upper bound version?

executorch/pyproject.toml

Lines 51 to 52 in 5ef38d3

	# Python dependencies required for use.
	requires-python = ">=3.10"

[metascroy]

What if we just hard enforce 3.12 as the upper bound version?

executorch/pyproject.toml

Lines 51 to 52 in 5ef38d3

	# Python dependencies required for use.
	requires-python = ">=3.10"

I think this is a good idea

[metascroy]

What if we just hard enforce 3.12 as the upper bound version?

executorch/pyproject.toml

Lines 51 to 52 in 5ef38d3

	# Python dependencies required for use.
	requires-python = ">=3.10"

I think this is a good idea

@jathu added 3.13 upper bound to pyproject.toml. Can you have a look?

[jathu]

I guess we should change this to sub 13

Suggested change

	requires-python = ">=3.10,<=3.12"
	requires-python = ">=3.10,<3.13"

[reneleonhardt]

Interestingly enough, I can install coremltools==8.3.0 and 9.0b1, but not executorch when I revert requires-python, so binary wheels from cibuildwheel would be greatly appreciated as always 😄

.venv/bin/python' '-c' 'import importlib.util; print(importlib.util.find_spec('torch').submodule_search_locations[0])'
      Error while adding torch to CMAKE_PREFIX_PATH. Exit code: 1
      Output:
      
      CMake Error at tools/cmake/Utils.cmake:135 (message):
        Error:
      
        Traceback (most recent call last):
      
          File "<string>", line 1, in <module>
            import importlib.util; print(importlib.util.find_spec('torch').submodule_search_locations[0])
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      
        AttributeError: 'NoneType' object has no attribute
        'submodule_search_locations'
      
      Call Stack (most recent call first):
        tools/cmake/Utils.cmake:109 (get_torch_base_path)
        CMakeLists.txt:470 (find_package_torch_headers)
      
      
      -- Configuring incomplete, errors occurred!

By the way, if I move up EXECUTORCH_ROOT_PATH in backends/apple/coreml/scripts/install_requirements.sh this seems to work to remove manual syncing:

COREMLTOOLS_VERSION="$(grep coremltools== $EXECUTORCH_ROOT_PATH/pyproject.toml | grep -Eo '[0-9]+\.[0-9]+[^ ;"]*')"