Skip to content

Change model1 #61

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Change model1 #61

wants to merge 2 commits into from

Conversation

Feng-ATM
Copy link

@Feng-ATM Feng-ATM commented Apr 3, 2023

.env.example

@mingyi77
Copy link

mingyi77 commented Apr 9, 2023

/gpt hello

@anc95
Copy link
Owner

anc95 commented May 26, 2023

the api key is expired, I just updated, please push new commit to test. enjoy

@ArnavK-09
Copy link

/gpt hello

Copy link

@bhattk64 bhattk64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Feng-ATM,

Thanks for the PR! I see this is a large refactor with a lot of changes, including:

  • Updates to the .env.example file with new variables.
  • Addition of a new WebAssembly binary file (_tiktoken_bg.wasm).
  • Major rewrite of action/index.cjs with tens of thousands of additions and deletions.

Could you please help by adding a summary to the PR description explaining:

  • The goals of this refactor/change.
  • Why the .wasm file was added and its purpose.
  • Any breaking changes or things maintainers/reviewers should focus on.

Also, it would be helpful to have some inline comments in the code for key parts of the new logic, to make reviewing easier.

Looking forward to your clarifications before diving deeper into the code!

Thanks 🙂

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.

5 participants