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

Fix elevenLabs config error #5131

Merged
merged 5 commits into from
Aug 25, 2023
Merged

Fix elevenLabs config error #5131

merged 5 commits into from
Aug 25, 2023

Conversation

dannaward
Copy link
Contributor

Background

Running speak mode with elevenLabs on not only stable branch but also master fails with a config error like below.

  File "/Users/danna/Desktop/Personal/Auto-GPT/autogpt/speech/eleven_labs.py", line 46, in _setup
    if config.elevenlabs_voice_2_id in voice_options:
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'Config' object has no attribute 'elevenlabs_voice_2_id'. Did you mean: 'elevenlabs_voice_id'?

I just followed the guide on docs, and encountered this issue, so I think either the docs or the code should be updated.

Changes

I saw the commit history in which the elevenlabs_voice_2_id was deleted on .env.template, thus instead of adding it again on config.py, I decided to remove elevenlabs_voice_2_id-related-codes on eleven_labs.py. (I assumed eleven_labs.py is the legacy one here)
It's now all synchronized and is not running into errors anymore.

Documentation

Test Plan

It worked fine on both my local environment and docker.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.
  • I have run the following commands against my code to ensure it passes our linters:
    black .
    isort .
    mypy
    autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests --in-place

@netlify
Copy link

netlify bot commented Aug 18, 2023

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit f08fc3f
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/64e899037930be000717ddcf

@Pwuts
Copy link
Member

Pwuts commented Aug 24, 2023

This can be merged as soon as you put back data/.keep

@dannaward
Copy link
Contributor Author

@Pwuts Updated!

@Pwuts Pwuts merged commit 24e2fb2 into Significant-Gravitas:master Aug 25, 2023
@Pwuts
Copy link
Member

Pwuts commented Aug 25, 2023

Thanks! :)

@Pwuts Pwuts added this to the v0.4.8 milestone Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants