jeshraghian / snntorch

Deep and online learning with spiking neural networks in Python
https://snntorch.readthedocs.io/en/latest/
MIT License
1.28k stars 217 forks source link

add py3.12 to testing matrix #265

Closed cclauss closed 7 months ago

cclauss commented 10 months ago

ERROR: Could not find a version that satisfies the requirement torch>=1.1.0 (from versions: none)

https://pypi.org/project/torch

jeshraghian commented 10 months ago

The 3.12 test is failing, I think it's because 3.12 is missing from tox.ini under [tox] and [gh-actions].

cclauss commented 10 months ago

Travis CI - Pull Request Expected — Waiting for status to be reported Required should probably be removed from the required tests below unless there is an active, paid subscription to Travis CI which is no longer free to open source projects.

cclauss commented 10 months ago

I think that adding py312 to tox.ini will not solve the problem which looks like this:

py: install_deps> python -I -m pip install -r /home/runner/work/snntorch/snntorch/requirements_dev.txt
    [ ... ]
        File "/tmp/pip-build-env-vfl9zw5b/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 20, in <module>
      ModuleNotFoundError: No module named 'imp'

https://docs.python.org/3.12/whatsnew/3.12.html#imp so setuptools in requirements_dev.txt needs to be upgraded.

cclauss commented 10 months ago
      if: matrix.python-version == '3.12'
      run: pip install --upgrade setuptools
jeshraghian commented 10 months ago

It looks like we're still running into the same error above.

cclauss commented 10 months ago

Please re-approve te workflow so we can see if it passes.

cclauss commented 9 months ago

https://pypi.org/project/torch is not yet compatible with the current version of Python

jeshraghian commented 7 months ago

We can open this back up once PyTorch catches up to Python 3.12.