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

feat: Config eternalai model from env #927

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

genesis-0000
Copy link
Contributor

Risks

Low

Background

What does this PR do?

Config eternalai model from env

What kind of change is this?

More flexible than with env

Why are we doing this? Any context or related work?

Currently, Eliza supports only OpenAI. However, centralized AI providers limit user control, pose trust issues, and create single points of failure with high costs and restricted access. Decentralized AI offers a more transparent, accessible, and resilient alternative, empowering users with greater autonomy.

Learn more about our decentralized inference AI: https://eternalai.org/api

Documentation changes needed?

My changes require a change to the project documentation.

Testing

Where should a reviewer start?

Reviewers start from .env.example

Detailed testing steps

Run pnpm start --characters="path/to/your/characters/eternal.character.json"

Deploy Notes

Database changes

No change

Deployment instructions

Change .env by adding more

ETERNALAI_MODEL=#require

Copy link
Collaborator

@monilpat monilpat left a comment

Choose a reason for hiding this comment

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

Thanks for doing this please address comment then good to go :)

Comment on lines 34 to 36
[ModelClass.SMALL]: settings.ETERNALAI_MODEL,
[ModelClass.MEDIUM]: settings.ETERNALAI_MODEL,
[ModelClass.LARGE]: settings.ETERNALAI_MODEL,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we add these existing values as backups if case nothing is configured?

Copy link
Contributor Author

@genesis-0000 genesis-0000 Dec 9, 2024

Choose a reason for hiding this comment

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

Yes. I did it
image

Thank for your comment sir!

Copy link
Collaborator

@monilpat monilpat left a comment

Choose a reason for hiding this comment

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

LGTM thanks for working on this :)

@jkbrooks jkbrooks merged commit 9b9b530 into elizaOS:main Dec 10, 2024
3 checks passed
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.

3 participants