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

new jukebox: high-definition music playing at lower quality #574

Closed
adamdmoss opened this issue Sep 16, 2017 · 5 comments
Closed

new jukebox: high-definition music playing at lower quality #574

adamdmoss opened this issue Sep 16, 2017 · 5 comments
Assignees
Labels
stale This label will be removed soon

Comments

@adamdmoss
Copy link

I have a bunch of 48KHz music, some 192KHz 24-bit etc.
According to my DAC's readout, it's being fed 44.1KHz/16-bit audio data by the new jukebox.

I could have sworn that an earlier version of the new jukebox branch was playing hi-def files at their native rate.

Did something change? Did I only hallucinate this working in the past? I may have screwed up my ALSA config and accidentally forcing 44.1/16 but I don't think so (also tried without an asound.conf at all).

@adamdmoss
Copy link
Author

I'm at f5454bb

@muff1nman
Copy link
Contributor

Assigned @biconou

@biconou
Copy link
Contributor

biconou commented Sep 23, 2017

@adamdmoss Can you help me investigate on that issue ?

Can you change the log level to DEBUG for package com.github.biconou.AudioPlayer ? You can achieve that in several ways depending how you run Airsonic.
Here is one suggested method (https://airsonic.github.io/docs/logging/)

Now inspect what appears in the airsonic.log file.

At jukebox player initialization you should see something like that.

2017-09-23 16:39:08.883  INFO --- o.a.p.service.JukeboxJavaService         : use mixer : CUBE [plughw:1,0]
2017-09-23 16:39:09.044  INFO --- c.g.b.AudioPlayer.JavaPlayer             : Player com.github.biconou.AudioPlayer.JavaPlayer@71e6fcd initialisation
2017-09-23 16:39:09.045  INFO --- c.g.b.AudioPlayer.JavaPlayer             : mixer is : CUBE [plughw:1,0] - Direct Audio Device: Audiotrak ProDigy CUBE, USB Audio, USB Audio
2017-09-23 16:39:09.079  INFO --- c.g.b.AudioPlayer.JavaPlayer             : Supported format : PCM_SIGNED 44100.0 Hz, 16 bit, stereo, 4 bytes/frame, little-endian
2017-09-23 16:39:09.098  INFO --- c.g.b.AudioPlayer.JavaPlayer             : Supported format : PCM_SIGNED 96000.0 Hz, 24 bit, stereo, 6 bytes/frame, little-endian
2017-09-23 16:39:09.099  INFO --- c.g.b.AudioPlayer.JavaPlayer             : Player com.github.biconou.AudioPlayer.JavaPlayer@71e6fcd initialized

In my case, we can see that my card number is 1 and it supports both normal and High resolution audio formats.

Now I play a hi resolution file, and I can see that in the log file.

2017-09-23 17:05:03.804 DEBUG --- c.g.b.AudioPlayer.JavaPlayer             : Player com.github.biconou.AudioPlayer.JavaPlayer@c89ba2a : Play
2017-09-23 17:05:03.818 DEBUG --- c.g.b.AudioPlayer.JavaPlayer             : Raw format : PCM_SIGNED 96000.0 Hz, 24 bit, stereo, 6 bytes/frame, little-endian (naim-test-2-wav-24-96000.wav)
2017-09-23 17:05:03.826  INFO --- o.a.p.service.JukeboxJavaService         : [onSongStart] admin starting jukebox for "WAV/naim-test-2-wav-24-96000.wav"
2017-09-23 17:05:03.843 DEBUG --- c.g.b.AudioPlayer.JavaPlayer             : Audio Format is : PCM_SIGNED 96000.0 Hz, 24 bit, stereo, 6 bytes/frame, little-endian 

While playing, if I use a terminal and type this

cd /proc/asound/card1/pcm0p/sub0

cat hw_params

I can see that.

access: RW_INTERLEAVED
format: S24_3LE
subformat: STD
channels: 2
rate: 96000 (96000/1)
period_size: 1920
buffer_size: 48000

Can you do the same and it will help me understand what goes wrong ?

biconou pushed a commit to biconou/airsonic that referenced this issue Oct 17, 2017
@biconou
Copy link
Contributor

biconou commented Oct 17, 2017

@adamdmoss your contribution for testing #600 PR would be great.

biconou pushed a commit to biconou/airsonic that referenced this issue Jan 20, 2018
@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
@stale stale bot closed this as completed Jan 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale This label will be removed soon
Projects
None yet
Development

No branches or pull requests

4 participants