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

tox.ini: Enable Python 3.13 #646

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Conversation

bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented Oct 17, 2024

What does this Pull Request accomplish?

Enable running system tests with Python 3.13.

Why should this Pull Request be merged?

Finish fixing #644

What testing has been done?

None

@bkeryan
Copy link
Collaborator Author

bkeryan commented Oct 17, 2024

The py64 runner has been updated but the py32 runner has not.

Copy link

Test Results

    38 files  +    2      38 suites  +2   58m 32s ⏱️ + 5m 16s
 2 348 tests ±    0   1 994 ✅ ±    0    354 💤 ±  0  0 ❌ ±0 
46 232 runs  +3 540  39 434 ✅ +2 994  6 798 💤 +546  0 ❌ ±0 

Results for commit e215298. ± Comparison against base commit ee791a3.

tox.ini Show resolved Hide resolved
@bkeryan
Copy link
Collaborator Author

bkeryan commented Oct 17, 2024

I double-checked the output to make sure it's actually running 3.13.

py32: https://github.com/ni/nidaqmx-python/actions/runs/11376831246/job/31650061261?pr=646

py313-base: skipped because could not find python interpreter with spec(s): py313
py313-base: SKIP ⚠ in 2.42 seconds

This is due to the fact that creating an updated py32 image failed.

py64: https://github.com/ni/nidaqmx-python/actions/runs/11376831246/job/31650061486?pr=646

py313-grpc: commands[0]> poetry run python --version
Python 3.13.0

py64 works and py32 fails gracefully, so I guess this PR doesn't have to wait for the new py32 image. I will manually run the tests on 32-bit Python, though.

@bkeryan
Copy link
Collaborator Author

bkeryan commented Oct 17, 2024

PS D:\dev\nidaqmx-python> poetry run pytest -v
=================================================== test session starts ===================================================
platform win32 -- Python 3.13.0, pytest-8.3.3, pluggy-1.5.0 -- D:\dev\nidaqmx-python\.venv\Scripts\python.exe
...
================================================= short test summary info =================================================
FAILED tests/component/system/test_device_properties.py::test___ext_cal_last_date_and_time___no_errors[library_init_kwargs] - assert 2009 > 2009
FAILED tests/component/system/test_device_properties.py::test___ext_cal_last_date_and_time___no_errors[grpc_init_kwargs] - assert 2009 > 2009
=========================== 2 failed, 2286 passed, 48 skipped, 11 xfailed in 212.89s (0:03:32) ============================

@bkeryan bkeryan merged commit dab722c into master Oct 17, 2024
19 checks passed
@bkeryan bkeryan deleted the users/bkeryan/python-3.13-system-tests branch October 17, 2024 17:20
bkeryan added a commit that referenced this pull request Oct 17, 2024
(cherry picked from commit dab722c)
bkeryan added a commit that referenced this pull request Oct 17, 2024
* nidaqmx: Fix installation with Python 3.13 (#645)

* tests: Fix OverflowError: Python integer 36767 out of bounds for int16

* tests: Skip more examples

* nidaqmx: Update dependency versions for Python 3.13 support

* Update poetry.lock

* GitHub: Enable unit tests on Python 3.13

* tests: Reformat with black

* tests: Fix E721 do not compare types, for exact checks use `is` / `is not`, for instance checks use `isinstance()`

* nidaqmx: Allow newer types-protobuf

* nidaqmx: Update poetry.lock

(cherry picked from commit ee791a3)

* tox: Enable Python 3.13 (#646)

(cherry picked from commit dab722c)

* nidaqmx: Add Python 3.13 trove classifier and update changelog (#649)

* nidaqmx: Add Python 3.13 trove classifier

* nidaqmx: Update changelog

(cherry picked from commit ba8a962)
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.

2 participants