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

Mac client example #123

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Mac client example #123

wants to merge 2 commits into from

Conversation

SlavikCA
Copy link

No description provided.

@SlavikCA
Copy link
Author

Tested it on my MacBook M1 Pro laptop.

I actually recorded the demo with it: https://youtu.be/kP4v07AlTow

On that demo - I speak Russian language and have it all translated to English.

README.md Outdated

Live stream raw audio:
```
ffmpeg -hide_banner -f avfoundation -i ":0" -ac 1 -ar 48000 -acodec pcm_s16le -ar 16000 -f s16le -loglevel error - | nc localhost 50000
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks.

-ar 48000 seems to be redundant, because there is -ar 16000.

And -acodec pcm_s16le and -f s16le seem to be duplicated.

Can you check if these options could be simpler?

Also, can you expand what to do with list of microphones? You probably need to pass the :0 from the list to the other command, right? Make it explicit.

README.md Outdated
@@ -208,6 +208,17 @@ arecord -f S16_LE -c1 -r 16000 -t raw -D default | nc localhost 43001

- nc is netcat with server's host and port

#### Client example on the Mac system:

List microphones:
Copy link
Collaborator

Choose a reason for hiding this comment

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

-> List of sounddevices

not only microphones are there

@Gldkslfmsd
Copy link
Collaborator

I actually recorded the demo with it: https://youtu.be/kP4v07AlTow

On that demo - I speak Russian language and have it all translated to English.

Thanks! It's very nice!

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.

2 participants