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

Merge In Recent Fixes #14

Merged
merged 14 commits into from
Jun 7, 2024
Merged

Merge In Recent Fixes #14

merged 14 commits into from
Jun 7, 2024

Conversation

macwilk
Copy link
Collaborator

@macwilk macwilk commented Jun 7, 2024

Merges in some fixes we've made in the last 48 hrs focused around modernizing and improving the usability of our quickstarts.

This should mean you don't need to install extras to get quick starts to work, poetry install vocode should be all thats needed :)

@macwilk macwilk requested review from ajar98 and adnaans June 7, 2024 16:26
Copy link
Owner

@ajar98 ajar98 left a comment

Choose a reason for hiding this comment

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

Looks like is failing lint for something we should have caught in the internal repo, yeesh

mypy!!!

@macwilk macwilk force-pushed the mac/merge-fixes branch from 395b308 to 4c57009 Compare June 7, 2024 16:32
@macwilk
Copy link
Collaborator Author

macwilk commented Jun 7, 2024

Looks like is failing lint for something we should have caught in the internal repo, yeesh

mypy!!!

never would have caught me saying this before but can't wait till this becomes a submodule internally so we dont have any more mypy suprises :(

@macwilk macwilk requested a review from ajar98 June 7, 2024 16:37
@macwilk macwilk merged commit cfd6226 into main Jun 7, 2024
2 checks passed
@macwilk macwilk deleted the mac/merge-fixes branch June 7, 2024 16:46
ajar98 added a commit that referenced this pull request Jun 14, 2024
* Update Readme with Preview Info (#1)

* Update Readme with Preview Info

* We're not quite that far along

* Update Structure to be more Pleasing to the Eyes

* Add changelog to readme

---------

Co-authored-by: srhinos <[email protected]>
Co-authored-by: Adnaan Sachidanandan <[email protected]>

* The Big Diff (#2)

* The Big Diff

* remove tests on 3.8 and 3.9

* Update README.md

* Update README.md

* fix turn based quickstart (#3)

* [hotfix] remove unused import (#4)

* Update README.md

* Update README.md

* Remove create_speech() from rime synthesizer (#6)

* Fix default factory for elevenlabs WS (#12)

* dispatch into elvenlabsws if experimental_websocket is on

* fix mypy

* Merge In Recent Fixes (#14)

* [docs sprint] Updates docs for using transcribers (#9)

* [docs sprint] phrase trigger documentation (#16)

* [docs sprint] update open source quickstarts (#15)

* [docs sprint] Add Documentation on Using Vocode's Loguru Implementation (#19)

* [docs sprint] Add Documentation on Using Vocode's Loguru Implementation

* Remove Tracing

---------

Co-authored-by: srhinos <[email protected]>

* [docs sprint] Updates docs for using synthesizers (#8)

* [docs sprint] using synthesizers docs update

* update docs for elevenlabs ws

* Apply suggestions from code review

Co-authored-by: Adnaan Sachidanandan <[email protected]>

---------

Co-authored-by: Adnaan Sachidanandan <[email protected]>

* [docs sprint] Updates docs for react quickstart (#10)

* [docs sprint] Updates docs for react quickstart

* PR feedback

* changes azure to override create_speech_uncached (#21)

* [docs sprint] Adds docs for conversation mechanics and moves endpointing docs from transcribers (#11)

* [docs sprint] Updates docs for using transcribers

* Adds docs for conversation mechanics and moves endpointing docs from transcribers

* Update docs/open-source/conversation-mechanics.md

Co-authored-by: Adnaan Sachidanandan <[email protected]>

* use mdx

* PR feedback

---------

Co-authored-by: Adnaan Sachidanandan <[email protected]>

* updates docs for events manager (#7)

* add cartesia synthesizer (#17)

* add cartesia synthesizer

* make Cartesia dependency optional, add it to the synthesizers extra group

* lazy import cartesia

* improved lazy loading, and added api_key as a config parameter

* improvements to cartesia synth
* use create_speech_uncached
* use existing abstractions default encoding and sample rates

* Remove redundant api_key assignment

Co-authored-by: Ajay Raj <[email protected]>

* Remove default setting of sampling rate

Co-authored-by: Ajay Raj <[email protected]>

* Remove default setting of audio_encoding

Co-authored-by: Ajay Raj <[email protected]>

* remove default setting of sampling rate

Co-authored-by: Ajay Raj <[email protected]>

* Remove redundant setting of audio enconding

the output device handles this

Co-authored-by: Ajay Raj <[email protected]>

* build failed with poetry.lock file. re-updating it

---------

Co-authored-by: Ajay Raj <[email protected]>

* Unset docs / README changes

* Unset docs changes (cont.)

* unset poetry version change

* update poetry.lock

---------

Co-authored-by: Mac Wilkinson <[email protected]>
Co-authored-by: srhinos <[email protected]>
Co-authored-by: Adnaan Sachidanandan <[email protected]>
Co-authored-by: rjheeta <[email protected]>
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