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

updates the modelClass to a valid value #169

Closed
wants to merge 1 commit into from

Conversation

sirkitree
Copy link
Collaborator

@sirkitree sirkitree commented Nov 2, 2024

also adds some debug into generation regarding which module is in use

Resolves #168

Relates to:

#168

Risks

  • medium, can effect discord use

Background

What does this PR do?

  • This passes the ModelClass instead of an incorrect string value

What kind of change is this?

Bug fixes (non-breaking change which fixes an issue)

Testing

Detailed testing steps

  1. Start your bot before the code change, using
    "modelProvider": "openai",
    ... notice the error in console if you address your bot.
  2. Apply the patch.
  3. Restart your bot and notice it parses the message properly.

- adds some debug into generation regarding which module is in use

Resolves #168
@sirkitree
Copy link
Collaborator Author

nevermind, looks like this was fixed in between my first pull and debugging

@sirkitree sirkitree closed this Nov 2, 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.

TypeError: Cannot read properties of null (reading 'queueTextCompletion')
1 participant