-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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: missing updates for logger.ts #525
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work - please remove the change in defaultCharacter and get the CI / CD pipeline working :)
@@ -5,7 +5,7 @@ export const defaultCharacter: Character = { | |||
username: "eliza", | |||
plugins: [], | |||
clients: [], | |||
modelProvider: ModelProviderName.OPENAI, | |||
modelProvider: ModelProviderName.OLLAMA, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should remove this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm updating it back to OPENAI as it was.
working on it |
done |
This comment was marked as abuse.
This comment was marked as abuse.
@monilpat I've made the requested fixes/updates. It's ready to be merged. |
minor conflict @yodamaster726 @monilpat |
Somehow I missed this file update on my last PR.