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

State that Python 3.12 isn't working right now #218

Closed
CRD716 opened this issue Oct 14, 2023 · 3 comments · Fixed by #221
Closed

State that Python 3.12 isn't working right now #218

CRD716 opened this issue Oct 14, 2023 · 3 comments · Fixed by #221

Comments

@CRD716
Copy link
Contributor

CRD716 commented Oct 14, 2023

psycopg/psycopg2#1628

python db_init.py currently fails on python 3.12 with

Traceback (most recent call last):
  File "E:\Installs\gpt-pilot\pilot\db_init.py", line 3, in <module>
    from database.database import create_tables, drop_tables
  File "E:\Installs\gpt-pilot\pilot\database\__init__.py", line 1, in <module>
    from .database import database_exists, create_database, save_app
  File "E:\Installs\gpt-pilot\pilot\database\database.py", line 6, in <module>
    import psycopg2
  File "E:\Installs\gpt-pilot\pilot-env\Lib\site-packages\psycopg2\__init__.py", line 51, in <module>
    from psycopg2._psycopg import (                     # noqa
ImportError: DLL load failed while importing _psycopg: The specified module could not be found.
@CRD716
Copy link
Contributor Author

CRD716 commented Oct 14, 2023

Maybe consider upgrading to psycopg3?

@Umpire2018
Copy link
Contributor

08280d4

Confimed.
It seems that python 3.12 doesn't work at the moment but thanks for pointing out.Please open a PR of update required python version in README.MD if you can.

@Umpire2018
Copy link
Contributor

About #221 (comment)

I believe there is no need to add rust to requirements.txt because pip install tiktoken will finish the job. What tiktoken rely is theirs concern.

Maybe you can find help from openai/tiktoken#172 .

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