pipecat icon indicating copy to clipboard operation
pipecat copied to clipboard

Add vision (image description)

Open chadbailey59 opened this issue 11 months ago • 6 comments

chadbailey59 avatar Mar 18 '24 22:03 chadbailey59

...aaand we've got our first autopep8 fight. I can see the changes that the linter wants, but even when I explicitly "format document" with autopep8 in VS code, it doesn't make those changes. Is there a secret autopep8 config file somewhere?

chadbailey59 avatar Mar 19 '24 03:03 chadbailey59

My 2 suggestions for 12-describe-video.py

  1. remove this line: from dailyai.services.deepgram_ai_services import DeepgramTTSService since you aren't using Deepgram TTS in this example

  2. Either use Speech-to-text explicitly and request the right API key for it, or set start_transcription=False. Otherwise, a user gets this cryptic error: 40 2024-03-19 01:19:49,832 Transcription error: Either set "deepgram:DEEPGRAM_API_KEY" at domain/room config or contact support to enable transcription for your domain

philtsip avatar Mar 19 '24 05:03 philtsip

...aaand we've got our first autopep8 fight. I can see the changes that the linter wants, but even when I explicitly "format document" with autopep8 in VS code, it doesn't make those changes. Is there a secret autopep8 config file somewhere?

Sorry @chadbailey59. I'll see if I can figure it out with VSCode

aconchillo avatar Mar 19 '24 15:03 aconchillo

@chadbailey59 Configure your VSCode autopep8 extension with these arguments:

image

Also, enable formatting on save (check the extension documentation for how to do it).

aconchillo avatar Mar 19 '24 15:03 aconchillo

I'll update the hacking section of the README with these VS Code requirements.

aconchillo avatar Mar 19 '24 15:03 aconchillo

I'll update the hacking section of the README with these VS Code requirements.

See https://github.com/daily-co/daily-ai-sdk/pull/78

aconchillo avatar Mar 19 '24 17:03 aconchillo

This PR is now obsolete. Closing.

aconchillo avatar Aug 28 '24 04:08 aconchillo