Pin max Python version to 3.12 on MacOS

[JCGoran]

Should fix failures on MacOS 13 such as https://github.com/neuronsimulator/nrn-build-ci/actions/runs/11546453401/job/32134846694

[JCGoran]

Ignore the 8.2.6 failure, we should be using the pre-built wheels from PyPI for that version anyway.

[1uc]

This is one error I found in the logs:

 Traceback (most recent call last):
  File "/Users/runner/work/nrn-build-ci/nrn-build-ci/nrn/setup.py", line 659, in <module>
    mac_osx_setenv()
  File "/Users/runner/work/nrn-build-ci/nrn-build-ci/nrn/setup.py", line 628, in mac_osx_setenv
    if py_osx_framework is not None and explicit_target > py_osx_framework:
                                        ^^^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'explicit_target' where it is not associated with a value

Is it directly related to this PR?

[JCGoran]

This is one error I found in the logs:

 Traceback (most recent call last):
  File "/Users/runner/work/nrn-build-ci/nrn-build-ci/nrn/setup.py", line 659, in <module>
    mac_osx_setenv()
  File "/Users/runner/work/nrn-build-ci/nrn-build-ci/nrn/setup.py", line 628, in mac_osx_setenv
    if py_osx_framework is not None and explicit_target > py_osx_framework:
                                        ^^^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'explicit_target' where it is not associated with a value

Is it directly related to this PR?

See #74 (comment), we aren't going to make another patch release of NEURON 8.2.x so I don't think it's important.

[nrnhines]

we aren't going to make another patch release of NEURON 8.2.x so I don't think it's important.

You never know... But can you remind me why Python3.13 can't easily work for NEURON 8.2?

[JCGoran]

we aren't going to make another patch release of NEURON 8.2.x so I don't think it's important.

You never know... But can you remind me why Python3.13 can't easily work for NEURON 8.2?

Regarding the release: good point!

Regarding 3.13: I was doing a bunch of testing earlier in neuronsimulator/nrn#3103, and the two issues that arose were:

• MUSIC doesn't work on Python3.13 since it's not compatible OOTB with mpi4py 4 (mpi4py < 4 will not have releases for 3.13); not really a blocker though since we don't ship it in wheels anyway
• a bunch of tests fail due to TSAN complaining on both MacOS and Ubuntu: https://github.com/neuronsimulator/nrn/actions/runs/11179005928/job/31077747315?pr=3103, though, oddly enough, only on certain versions (MacOS 14 and ubuntu 24)

[nrnhines]

Thanks for the informative comment #74 (comment)

This will be easy to revert in the future if it becomes desirable. And we can work on the thread-sanitizer issue at that time.