Add audio normalization device config option (#255)

- Add `normalization` device config option for audio normalization - Update device default config Co-authored-by: Thang Pham <[email protected]>
aome510 · Sep 22, 2023 · d06ec2a · d06ec2a
1 parent 05fbda6
commit d06ec2a
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 11 deletions.
diff --git a/docs/config.md b/docs/config.md
@@ -124,13 +124,14 @@ esac
 
 The configuration options for the [Librespot](https://github.com/librespot-org/librespot) integrated device are specified under the `[device]` section in the `app.toml` file:
 
-| Option        | Description                                                             | Default          |
-| ------------- | ----------------------------------------------------------------------- | ---------------- |
-| `name`        | The librespot device's name                                             | `spotify-player` |
-| `device_type` | The librespot device's type                                             | `speaker`        |
-| `volume`      | Initial volume (in percentage) of the device                            | `50`             |
-| `bitrate`     | Bitrate in kbps (`96`, `160`, or `320`)                                 | `160`            |
-| `audio_cache` | Enable caching audio files (store in `$APP_CACHE_FOLDER/audio/` folder) | `false`          |
+| Option          | Description                                                             | Default          |
+| --------------- | ----------------------------------------------------------------------- | ---------------- |
+| `name`          | The librespot device's name                                             | `spotify-player` |
+| `device_type`   | The librespot device's type                                             | `speaker`        |
+| `volume`        | Initial volume (in percentage) of the device                            | `70`             |
+| `bitrate`       | Bitrate in kbps (`96`, `160`, or `320`)                                 | `320`            |
+| `audio_cache`   | Enable caching audio files (store in `$APP_CACHE_FOLDER/audio/` folder) | `false`          |
+| `normalization` | Enable audio normalization                                              | `false`          |
 
 More details on the above configuration options can be found under the [Librespot wiki page](https://github.com/librespot-org/librespot/wiki/Options).
 

diff --git a/examples/app.toml b/examples/app.toml
@@ -28,6 +28,7 @@ playback_window_width = 6
 [device]
 name = "spotify-player"
 device_type = "speaker"
-volume = 50
-bitrate = 160
+volume = 70
+bitrate = 320
 audio_cache = false
+normalization = false
diff --git a/spotify_player/src/config/mod.rs b/spotify_player/src/config/mod.rs
@@ -119,6 +119,7 @@ pub struct DeviceConfig {
     pub volume: u8,
     pub bitrate: u16,
     pub audio_cache: bool,
+    pub normalization: bool,
 }
 
 #[derive(Debug, Deserialize, Serialize, ConfigParse, Clone)]
@@ -258,9 +259,10 @@ impl Default for DeviceConfig {
         Self {
             name: "spotify-player".to_string(),
             device_type: "speaker".to_string(),
-            volume: 50,
-            bitrate: 160,
+            volume: 70,
+            bitrate: 320,
             audio_cache: false,
+            normalization: false,
         }
     }
 }

diff --git a/spotify_player/src/streaming.rs b/spotify_player/src/streaming.rs
@@ -174,6 +174,7 @@ pub fn new_connection(
             .to_string()
             .parse::<Bitrate>()
             .unwrap_or_default(),
+        normalisation: device.normalization,
         ..Default::default()
     };