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

GitHub Actions: Run unit tests on Python 3.13 #622

Draft
wants to merge 8 commits into
base: development
Choose a base branch
from

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Oct 23, 2024

@cclauss cclauss changed the title GitHub Actions: Run unit test on Python 3.13 GitHub Actions: Run unit tests on Python 3.13 Oct 23, 2024
Copy link

github-actions bot commented Oct 23, 2024

Binder 👈 Launch a binder notebook on this branch for commit 8178369

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 6e19470

Binder 👈 Launch a binder notebook on this branch for commit 2264106

Binder 👈 Launch a binder notebook on this branch for commit 9432621

Binder 👈 Launch a binder notebook on this branch for commit 07afaea

Binder 👈 Launch a binder notebook on this branch for commit 2cfaf33

Binder 👈 Launch a binder notebook on this branch for commit b404fe5

Binder 👈 Launch a binder notebook on this branch for commit c1688d3

Copy link

codecov bot commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.77%. Comparing base (4eb7aaf) to head (c1688d3).

❗ There is a different number of reports uploaded between BASE (4eb7aaf) and HEAD (c1688d3). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (4eb7aaf) HEAD (c1688d3)
3 2
Additional details and impacted files
@@               Coverage Diff               @@
##           development     #622      +/-   ##
===============================================
- Coverage        71.78%   66.77%   -5.02%     
===============================================
  Files               38       36       -2     
  Lines             3137     3079      -58     
  Branches           426      426              
===============================================
- Hits              2252     2056     -196     
- Misses             775      936     +161     
+ Partials           110       87      -23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cclauss cclauss marked this pull request as draft October 23, 2024 16:59
@weiji14
Copy link
Member

weiji14 commented Oct 23, 2024

Waiting on numba/numba#9413, which in turn is waiting on llvmlite upstream.

Collecting numba (from datashader->icepyx==1.3.1.dev10+g16b2ffd)
  Downloading numba-0.60.0.tar.gz (2.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.7/2.7 MB 28.2 MB/s eta 0:00:00
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'error'
  error: subprocess-exited-with-error
  
  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [24 lines of output]
      Traceback (most recent call last):
        File "/opt/hostedtoolcache/Python/3.13.0/x64/lib/python3.13/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
          ~~~~^^
        File "/opt/hostedtoolcache/Python/3.13.0/x64/lib/python3.13/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ~~~~^^^^^^^^^^^^^^^^^^^^^^^^
        File "/opt/hostedtoolcache/Python/3.13.0/x64/lib/python3.13/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
        File "/tmp/pip-build-env-w55jox8j/overlay/lib/python3.13/site-packages/setuptools/build_meta.py", line 332, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=[])
                 ~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-w55jox8j/overlay/lib/python3.13/site-packages/setuptools/build_meta.py", line 302, in _get_build_requires
          self.run_setup()
          ~~~~~~~~~~~~~~^^
        File "/tmp/pip-build-env-w55jox8j/overlay/lib/python3.13/site-packages/setuptools/build_meta.py", line 516, in run_setup
          super().run_setup(setup_script=setup_script)
          ~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-w55jox8j/overlay/lib/python3.13/site-packages/setuptools/build_meta.py", line 318, in run_setup
          exec(code, locals())
          ~~~~^^^^^^^^^^^^^^^^
        File "<string>", line 51, in <module>
        File "<string>", line 48, in _guard_py_ver
      RuntimeError: Cannot install on Python version 3.13.0; only versions >=3.9,<3.13 are supported.
      [end of output]

@cclauss
Copy link
Contributor Author

cclauss commented Jan 9, 2025

Blocked waiting for numba>=0.61.0... Tests pass on release candidate 2.

requirements.txt: numba==0.61.0rc2; python_version == '3.13' # FIX ME!!!

@JessicaS11
Copy link
Member

@cclauss Should we go ahead and merge with your fix, or would it be better to wait for numba>=0.61.0?

@cclauss
Copy link
Contributor Author

cclauss commented Jan 9, 2025

We can wait... We should not use a dependency's pre-release version in production.

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.

3 participants