Skip to content
This repository has been archived by the owner on Sep 8, 2021. It is now read-only.

Downsampling of music in the web app doesn't work #624

Closed
KingDuckZ opened this issue Nov 10, 2017 · 8 comments
Closed

Downsampling of music in the web app doesn't work #624

KingDuckZ opened this issue Nov 10, 2017 · 8 comments
Labels
neverstale This label will be removed soon

Comments

@KingDuckZ
Copy link

Problem description

In the Transcoding tab I've put the following settings:
name: vorbis audio
convert from: mp3 oga aac m4a flac wav wma aif aiff ape mpc shn
convert to: ogg
step 1: ffmpeg -i %s -map 0:0 -b:a %bk -v 0 -f ogg -acodec libvorbis -

and

downsample command: ffmpeg -i %s -map 0:0 -b:a %bk -v 0 -f ogg -acodec libvorbis -

but during playback in the web app these settings seem to be ignored.

Steps to reproduce

  • add some high quality song to your library (I added a 256k vbr ogg created with vorbit tools/soundkonverter)
  • go to your player settings in your Airsonic web app and set the max bitrate to 128k, for example
  • save, navigate back to your song and play it
  • in the playlist at the bottom of the web app, note how big the file being played is (for example 10 MiB)
  • you can double check the file size on your server
  • now go to the Status page, under the Transmitted column see how the full 10 MiB get transmitted
  • if the song was actually downsampled to 128k, I'd expect 4 or 5 MiB to be transmitted instead

System information

  • Airsonic version: 10.0.1
  • Operating system: ArchLinux-ARM on Odroid-C2
  • Proxy server: tested through nginx/https only
  • Client: Firefox 52.4 on Gentoo Linux

Additional notes

As discussed on IRC

@heyarne
Copy link
Contributor

heyarne commented Nov 10, 2017

Are you sure this is an airsonic problem? Your config looks like it's not matching the .ogg:

convert from: mp3 oga aac m4a flac wav wma aif aiff ape mpc shn

@KingDuckZ
Copy link
Author

Heyame is the conversion setting also used for downsampling then?
What I did when I changed that setting was simply swap mp3/ogg between the convert to/convert from columns. So maybe the original settings work with ogg but not mp3?
I can't test mp3 because of bug #622.

@heyarne
Copy link
Contributor

heyarne commented Nov 10, 2017

To be fair: Without checking the code I don't see why conversion and downsampling shouldn't be the same thing. I always read it as "when bitrate is above X and the file extension matches one of the following Ys, execute this command and stream the result".

@KingDuckZ
Copy link
Author

I have no idea. On IRC they told me transcoding != downsampling, that's all I know. Oh and that downsampling is broken for vbr music, iirc. I'm not excluding that there could be mistakes in my config, that's why I posted it, but given what I've changed it feels like that setting would either break downsampling or transcoding. Like, say I add ogg to the "convert from" list, then would it transcode all 128k ogg to 128k ogg too? simply because it matches?

@jooola jooola added the issue label Nov 11, 2017
@stale
Copy link

stale bot commented Jul 29, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale This label will be removed soon label Jul 29, 2018
@KingDuckZ
Copy link
Author

What's with this idiotic bot? is this a microsoft thing?

@stale stale bot removed the stale This label will be removed soon label Jul 29, 2018
@jooola
Copy link
Contributor

jooola commented Jul 29, 2018

@KingDuckZ Nope, I added it to shake things a bit. Sorry if it generates a mess

@stale
Copy link

stale bot commented Oct 27, 2018

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale This label will be removed soon label Oct 27, 2018
@muff1nman muff1nman added the neverstale This label will be removed soon label Dec 3, 2018
@stale stale bot removed the stale This label will be removed soon label Dec 3, 2018
eharris added a commit to eharris/airsonic that referenced this issue Jun 18, 2019
eharris added a commit to eharris/airsonic that referenced this issue Jun 29, 2019
My guess is that this probably resolves airsonic#624.
eharris added a commit to eharris/airsonic that referenced this issue Jul 20, 2019
My guess is that this probably resolves airsonic#624.
eharris added a commit to eharris/airsonic that referenced this issue Dec 17, 2019
My guess is that this probably resolves airsonic#624.
eharris added a commit to eharris/airsonic that referenced this issue Mar 2, 2020
My guess is that this probably resolves airsonic#624.
eharris added a commit to eharris/airsonic that referenced this issue Apr 14, 2020
My guess is that this probably resolves airsonic#624.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
neverstale This label will be removed soon
Projects
None yet
Development

No branches or pull requests

5 participants