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

fix for python 3.9 import crewai error [#215] #693

Closed
wants to merge 1 commit into from

Conversation

lugq1990
Copy link
Contributor

This is used to fix for the python 1.10 lower version to use crewai error with:

TypeError: unsupported operand type(s) for |: 'type' and 'NoneType' in rpm_controller.py 

With the change from | to Union type just alike: max_rpm: Union[int, None] = Field(default=None) attr.

@lugq1990
Copy link
Contributor Author

@joaomdmoura could you please have a look for this? related to the lower python support from the type checking. Just keep consistency for the code side

@theCyberTech
Copy link
Contributor

theCyberTech commented Jun 13, 2024

You should to use Python 3.10 or higher 3.9 is not supported, what is the reason you would need to use 3.9?

Doesn't make sense to try and fix this in my opinion?

@lugq1990
Copy link
Contributor Author

You should to use Python 3.10 or higher 3.9 is not supported, what is the reason you would need to use 3.9?

Doesn't make sense to try and fix this in my opinion?

For me, currently base python version is 3.9, as for the framework side shouldn't just support with over 3.10, right?

@theCyberTech
Copy link
Contributor

So two options,

  1. Upgrade your base version to 3.10+
  2. Run your crew in a virtual env using pyenv, poetry or conda

@theCyberTech
Copy link
Contributor

https://docs.crewai.com/how-to/Installing-CrewAI/

This link states the Python versions CrewAI supports

@lugq1990
Copy link
Contributor Author

Sure as the document mentions that support with python 3.10 over, than just close this pr

@lugq1990 lugq1990 closed this Jun 17, 2024
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 this pull request may close these issues.

None yet

2 participants