AliceO2Group / o2tuner

A tool for optimizing routines in ALICE O2
GNU General Public License v3.0
4 stars 2 forks source link

Change python versions in tests #64

Closed benedikt-voelkel closed 16 hours ago

benedikt-voelkel commented 2 days ago

Remove 3.8 Keep 3.9 (used in alidist) Add 3.12 (current version)

benedikt-voelkel commented 2 days ago

For some reason it seems that (at least) the max-args=10 in the .pylint file is ignored. The other possibility is that .pylint is not even read for some reason. But right now I doubt the latter since in that case I would expect a lot more errors. I am looking into it.

Anyway, locally it works and it used to work in the CI as well.

benedikt-voelkel commented 17 hours ago

I see this https://github.com/pylint-dev/pylint/issues/9961 and this https://bugs.launchpad.net/neutron/+bug/2083227 The latter indeed seems to be the same issue to what we see here. I will force pylint for now to be version 3.2.7

I also checked locally with various different versions to verify this problem.

benedikt-voelkel commented 16 hours ago

Done 🥳 It is a bit dirty but maybe that is the best we can do for now...?!