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

nidaqmx doesn't support Python 3.13+ #644

Closed
zhindes opened this issue Oct 16, 2024 · 2 comments · Fixed by #645
Closed

nidaqmx doesn't support Python 3.13+ #644

zhindes opened this issue Oct 16, 2024 · 2 comments · Fixed by #645

Comments

@zhindes
Copy link
Collaborator

zhindes commented Oct 16, 2024

To reproduce:

  • Install Python 3.13
  • pip install nidaqmx
  • Run some of the shipping examples

At the very least I ran into this:

Traceback (most recent call last):
  File "C:\Users\User\dev\junyinta-nimbus-benchmark\ramp_up.py", line 5, in <module>
    import nidaqmx
  File "C:\Users\User\AppData\Local\pypoetry\Cache\virtualenvs\benchmark-PLHonyti-py3.13\Lib\site-packages\nidaqmx\__init__.py", line 3, in <module>
    from nidaqmx.scale import Scale
  File "C:\Users\User\AppData\Local\pypoetry\Cache\virtualenvs\benchmark-PLHonyti-py3.13\Lib\site-packages\nidaqmx\scale.py", line 3, in <module>
    import numpy
ModuleNotFoundError: No module named 'numpy'

That is because our numpy dependency excludes Python 3.13+ here. There may be other issues, though. We should officially support and test Python 3.13+

@bkeryan
Copy link
Collaborator

bkeryan commented Oct 16, 2024

Oops, I should have made that an open-ended range or capped it at 4.0 like I did for grpcio-tools: #453

As written, it means "we don't need numpy for 3.13 and later", which is incorrect.

@bkeryan
Copy link
Collaborator

bkeryan commented Oct 16, 2024

Note that we need to add Python 3.13 to the nidaqmxbot image build config.

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 a pull request may close this issue.

2 participants