Skip to content

Log verbosity, pid parametrised and pipe writing only on when sound is detected#19

Open
ale275 wants to merge 26 commits intob-fitzpatrick:masterfrom
ale275:master
Open

Log verbosity, pid parametrised and pipe writing only on when sound is detected#19
ale275 wants to merge 26 commits intob-fitzpatrick:masterfrom
ale275:master

Conversation

@ale275
Copy link

@ale275 ale275 commented Oct 12, 2025

Pull requests to incorporate different changes by different users

  1. Write to pipe only on sound (https://github.com/natedreger/cpiped) and (https://github.com/natedreger/NDpiped)
  2. Parametrized pid file by Mario Wenzel (https://github.com/maweki/cpiped)
  3. Log verbosity and other minor cleanups by me

ale275 and others added 22 commits October 12, 2025 13:57
- refactored val variable to samplerate
- made buffer and capture sizes calculation function of samplerate and captured channels count
- cleaned up help message in case of error
- made out pipe size twice the capture buffer size
- added error message when pipe resizing fails
- added warning when pipe size different from the requested one
…el count

- variable name for sample rate is CPIPED_SR
- variable name for sample size is CPIPED_SS
- variable name for channel count is CPIPED_CC
- those can be used by the called script to perform format conversion or other arithmetic if needed
@b-fitzpatrick
Copy link
Owner

@ale275 looks like a lot of good work. I haven't used this in years and don't have anything set up to test it. Would you be interested in taking over this repo?

@ale275
Copy link
Author

ale275 commented Dec 14, 2025

thank you @b-fitzpatrick. I have some more development for the cpiped ecosystem to develop so I am more than happy to support if you want

@b-fitzpatrick
Copy link
Owner

I looked into transferring the repo and decided against it. I'm thinking that I'll add a message pointing to the pull requests and archive the repo.

@ale275
Copy link
Author

ale275 commented Dec 15, 2025

Sure no problem

I will also rewrite the readme in the coming days to reference work done

thank you so much in the first place for this

@ale275 ale275 force-pushed the master branch 3 times, most recently from ec3f8e4 to 155b59e Compare December 28, 2025 20:00
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.

4 participants