chore: downgrade python requirement to 3.12#319
Conversation
Restores psycopg2 package to both project dependencies and pixi dependencies to enable PostgreSQL database connections with SQLAlchemy. The psycopg2-binary package was removed in the Python 3.12 downgrade, but is required for PostgreSQL support. Using psycopg2 from conda-forge instead of psycopg2-binary to ensure compatibility with pixi.
neomorphic
left a comment
There was a problem hiding this comment.
This breaks the production deployment as we need to have the psycopg2 package installed to use postgreSQL. I have pushed an update that restores it.
|
Thanks @allison-truhlar I tried a fresh checkout... (after renaming my previous Something isn't happy... |
|
However, upgrading to the pre-release version in my existing python 3.12 conda environment worked just fine... |
|
I wonder if my issues with |
|
@will-moore In your development set up - have you copied the server configuration file into the root of the project? This error suggests it's missing: I'm glad the pre-release is working for you on Python 3.12 |
|
@allison-truhlar - No I didn't copy over the config file. I don't think I needed to do that when I first ran the dev-launch a while back? But it's working fine now, thanks. |
|
@will-moore Good point re: the ordering of the development instructions - I'll move the copy config file instruction up. |
|
Also related: #324 |
This addresses issue #318
I noticed there were some unused dependencies listed under
[tool.pixi.dependencies], specifically,aiosqliteandpsycopg2-binary. I also removed the version requirements forhttpxandlxml, as these are not direct dependencies and the required versions resolved without the listed requirements. Removing these and downgrading the Python requirement to>=3.12.0from>=3.14.0, the application built fine and passed all tests. I manually tested the Okta login flow and it worked as well.I tried decreasing the Python minor version to
>=3.11.0, but x2s3 requires Python 3.12.*, so the dependencies could not resolve.Edit: ultimately, only
aiosqliteandlxmlwere removed from both the pypidependencieslist and the pixi env[tool.pixi.dependencies]list.