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.

Sign up for GitHub

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

Jump to bottom

Rewrite - Migrate from g4f to ChatGPT's proxy, utilising OpenAI's official api package #21

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Rewrite - Migrate from g4f to ChatGPT's proxy, utilising OpenAI's official api package #21

wants to merge 16 commits into from

Conversation

hakimifr
Copy link
Member

@hakimifr hakimifr commented Apr 14, 2024 •
edited by prathamdby
Loading

Progress Tracker

  • Migration of codebase

    • Add openai dependency
    • Remove traces of g4f and migrate to openai's package
    • Remove g4f dependency
    • Ensure consistency of lock file

  • Proxy server

    • Add the proxy server as git submodule
    • Change base URL of the openai package to the proxy's
    • Testing the server
      • Make sure completion work
      • Ensure no regression caused to the reply-tracker (context-keeping)

hakimifr and others added 4 commits April 14, 2024 01:38
@hakimifr
YeetAI: Track our own fork of the pyrogram's fork
752ce73 
* The fork gets updated frequently, and as
  a result, the poetry.lock's hash often get
  mismatched whenever it does. Tracking our
  own fork allows us to controllably sync
  changes and update the lock file accordingly.
@hakimifr
YeetAI: Add openai's official api package as dependency
cac175e
@prathamdby
src: Bot: Switch to a command-handler implementation
dcc8935 
Signed-off-by: Pratham Dubey <[email protected]>
@prathamdby
src: Bot: Implement category support for the command-handler
d7459b1 
Adapt to the logging format in line with the rest of the bot as well.

Signed-off-by: Pratham Dubey <[email protected]>
@prathamdby
src!: Migarate fully over to the official OpenAI SDK
c3f5e64 
Update the example .env file. Drop all of the G4F-related codebase. Clean up some parts of the codebase.

Signed-off-by: Pratham Dubey <[email protected]>
@prathamdby
src/commands/AI: ask: Switch to the async variant of the OpenAI client
11587cb 
Run the isort tool to sort out the project's import statements.

Signed-off-by: Pratham Dubey <[email protected]>
@hakimifr hakimifr added the enhancement Enhancement to existing features label Apr 15, 2024
@hakimifr
Add ChatGPT proxy's submodule
7e07785
@hakimifr
tests: Remove redundant g4f test since we stopped using it
10ae059
@hakimifr
src/commands: Add documentation on how to implement commands
801c30e
@hakimifr
src: Bot: Refactor the commands loader
3177854
@hakimifr
src: Nuke the (no longer used) module system/loader
702cfd9
@hakimifr
src: Bot: Move command loader to a new file (Command.py)
9079802
@hakimifr
src/commands/AI: ask: Nuke redundant event loop policy logging
5f1faee 
* Now that we no longer use g4f
@hakimifr
src: Command: Implement a more robust mechanism for command-files data
e90b731
@hakimifr
Update dependencies to fix GitHub's vulnerability warning
a0ac929
@hakimifr
Partial revert of commit: 'e90b731: src: Command: Implement a more ro…
c25d917 
…bust mechanism for command-files data

* Revert back to storing raw cmd data dict. Leave the class though,
  as it can be used to parse the data when needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pranayadmn pranayadmn

@hakimifr hakimifr

@prathamdby prathamdby

Labels
enhancement Enhancement to existing features
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hakimifr @pranayadmn @prathamdby