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

Update python to 3.10 to avoid error during gen_fcsv #15

Open
wants to merge 1 commit into
base: djay/docker
Choose a base branch
from

Conversation

mackenziesnyder
Copy link

  • updated the default python version to 3.10 to avoid error during the gen_fcsv rule
  • this aligns with the builder used in the docker file

@Dhananjhay
Copy link
Contributor

Can you share the error log and the command to reproduce the error? I suspect it might be coming from the tensorflow library. Removing support for all python versions except v3.10 might be too restrictive.

@ataha24
Copy link
Contributor

ataha24 commented Feb 12, 2025

Can you share the error log and the command to reproduce the error? I suspect it might be coming from the tensorflow library. Removing support for all python versions except v3.10 might be too restrictive.

I agree it may be too restrictive and not sure what the best way to move forward but here is my granular account.

  • First, is an issue of tensorflow on my local M2. Below is a screenshot of the error. I got this when running using new docker PR. I dont think python versioning may fix this completely and M2/apple sillicon could be something we support spearately?
Screenshot 2025-02-12 at 1 29 31 PM
  • The second issue which is more relevant to this PR and happens on CBS when using poetry. For Datrie to be configured correctly, we need poetry to be using python3.10. Below is a screenshot of the error when I use poetry to install autoafids (using docker PR).
Screenshot 2025-02-12 at 1 44 12 PM Screenshot 2025-02-12 at 1 44 26 PM

These two are slightly separate but I guess good to keep in mind as we move forward.

@Dhananjhay Dhananjhay added the bug Something isn't working label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants