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

Output as wav #193

Open
audiofool opened this issue Apr 1, 2022 · 6 comments
Open

Output as wav #193

audiofool opened this issue Apr 1, 2022 · 6 comments
Labels
enhancement New feature or request
Milestone

Comments

@audiofool
Copy link

I am having some success using camilladsp within LMS(slimserver) as part of my DSP upsampling chain. I pipe to camilladsp which then pipes to stdout. I currently have to add sox to the chain to convert from raw to wav format which LMS(squeezelite) requires. Is it possible to add a wav output option?

@HEnquist
Copy link
Owner

HEnquist commented Apr 2, 2022

Yes this would be quite straightforward. There is only one tricky part. The wav header contains the total length of the file, which isn't known when the header is written. It should be fine to set the length to the maximum possible value. And if that causes trouble then it would be possible to rewind and update the length before closing the file.

@HEnquist HEnquist added the enhancement New feature or request label Apr 2, 2022
@audiofool
Copy link
Author

In my current process with sox I include --ignore-length since I am not writing to a file but using stdout to squeezelite which I believe ignores length, so setting to max value should work.

@Foxenfurter
Copy link

Hi - I am continuing with my plugin work on LMS using CamillaDSP as discussed in another Issue. I have a basic implementation running with core features added and I am now trying to iron out the wrinkles (it is very wrinkly!). Like audiofool, I really need a wav output so I was wondering whether this feature is available?

I will assume it isn't and will add in the extra sox command for now.

@HEnquist HEnquist added this to the 2.0 milestone Jan 20, 2023
@HEnquist
Copy link
Owner

It's not done yet. I'll try to include it in the next version.

@Foxenfurter
Copy link

Thanks - no rush. I managed to add in a sox conversion within my embedded app and it seems to be working OK.

@HEnquist HEnquist modified the milestones: 2.0, 2.1 May 14, 2023
@HEnquist
Copy link
Owner

implemented in #325

@HEnquist HEnquist modified the milestones: 2.1, 3.0 Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants