Upgrade Python to 3.12

[annehaley]

No description provided.

[brianhelba]

LGTM, and it looks like scikit-learn is indeed at 1.8.0 now.

[brianhelba]

The test failure looks like a legitimate existing problem that is now newly detected by Python 3.12 (TemporaryFile not getting closed).

If you can investigate and fix, that'd be ideal. However, if it's coming from a dependency outside your control (and you can't fix it by just upgrading that dependency ), then you can add a Pytest warning filter to ignore it (which I'm happy to review too, if you'd like).

[annehaley]

Yeah, I'm trying to investigate the origin of the unclosed temp file. The only places we use the tempfile library are for creating TemporaryDirectory objects scoped via a with block. I was under the impression that the with context would clean up those files neatly. I'll keep researching the cause before we decide whether to ignore the warning.

[annehaley]

It took some hunting, but I did find a spot that needed a with block. I didn't see it at first because we don't explicitly invoke the tempfile library, but there was a spot where we open an S3FileField file without closing it. Fixed in c1a5f53.