Persistent volume between restarts for Onju Voice

[VivantSenior]

Added persistent (between device restarts and firmware updates) volume value. Please test as I am using MicroWakeWord flavor.

[jherby2k]

Doesn't work for me. Tried both a firmware update and a hard power cycle. Back to 100% each time.

Shouldn't you be storing it inside the set_volume function?

[VivantSenior]

Tried to put the logic in the set_volume but as far as I remember during my tests I noticed this function is not used when changing volume from Home Assistant UI. Please note that ESP saves variables to the memory at some interval (60s ?) therefor if you set new value and there is a is sudden loss of voltage it won't restore the newest value. It works when you upload new firmware or use Restart ESPHome function.

Try to add debug logs and paste it here.

logger:
  level: DEBUG

      - media_player.volume_set:
          id: onju_out
          volume: !lambda |-
            ESP_LOGD("persistent_volume", "--- volume_percent restored value: %f ---", id(volume_percent));
            return id(volume_percent);

media_player:
  - platform: adf_pipeline
    id: onju_out
    name: None
    internal: false
    keep_pipeline_alive: true
    pipeline:
      - self
      - resampler
      - adf_i2s_out
    on_state:
      then:
        - lambda: |-
            static float old_volume = -1;
            float new_volume = id(onju_out).volume;
            if(abs(new_volume-old_volume) > 0.0001) {
              if(old_volume != -1) {
                id(show_volume)->execute();
              }
            }
            old_volume = new_volume;
            ESP_LOGD("persistent_volume", "--- volume_percent before change: %f ---", id(volume_percent));
            id(volume_percent) = old_volume;
            ESP_LOGD("persistent_volume", "--- volume_percent after change: %f ---", id(volume_percent));

      - media_player.volume_set:
          id: onju_out
          volume: !lambda |-
            ESP_LOGD("persistent_volume", "--- volume_percent restored value: %f ---", id(volume_percent));
            return id(volume_percent);

[jherby2k]

Ah, issue seems to be here. I i just change this lambda to return id(volume_percent); it works, but i assume the clamping is there for a reason and I don't really understand this very well, so i'll let you fix it appropriately.

  - id: set_volume
    mode: restart
    parameters:
      volume: float
    then:
      - media_player.volume_set:
          id: onju_out
          volume: !lambda return clamp(id(onju_out).volume+volume, 0.0f, 1.0f);