Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sqlalchemy v2.0.0 bug causes cells.SingleCells to not work: ObjectNotExecutableError with read_sql #250

Closed
jenna-tomkinson opened this issue Feb 3, 2023 · 1 comment · Fixed by #251

Comments

@jenna-tomkinson
Copy link
Member

jenna-tomkinson commented Feb 3, 2023

I noticed that when I pip installed Pycytominer to use in a project yesterday that I was getting this error when trying to run cells.SingleCells from the cells.py file.

ObjectNotExecutableError: Not an executable object: 'select * from Per_Image'

This error is caused by a bug between Pandas and the newest release of Sqlalchemy from a few days ago. It is described further in this issue from the Pandas GitHub.

I noticed that in the requirements.txt that it states:

sqlalchemy>=1.3.6

This means it is installing the latest version of Sqlalchemy when pip installing. This means a new user if they wanted to use the cells.SingleCells function or any function in Pycytominer that uses Sqlalchemy read_sql function using a string, then it will not work.

The only work around I have that works is a conda environment that installs Sqlalchemy v1.4.42 (which is the last version that works) and pip installs Pycytominer. The conda install override the pip install.

Once this bug is fixed, then this won't be a worry, but I wanted to make this issue for others to be aware in case they get this same error.

@bethac07 bethac07 mentioned this issue Feb 6, 2023
10 tasks
@bethac07
Copy link
Member

bethac07 commented Feb 7, 2023

Thanks Jenna! Put in a PR to solve the immediate issue, and another issue to remind us to keep an eye out for the pandas fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants