Vost3 / hass-palazzetti

Manage your Palazzetti stove with home assistant
29 stars 16 forks source link

Documentation clarification #5

Closed tanc closed 2 years ago

tanc commented 3 years ago

First I want to thank you for taking the time to put this together, I was really happy to discover this integration.

I'm struggling to understand the documentation though, probably because I'm a beginner with Home Assistant.

In your docs there are the following headings, each with yaml to be added somewhere:

Configuration configuration.yaml Service Automation automation.yaml Sensor Template Script

It's not at all clear to me where to put the yaml in each section. It looks like Configuration and Service go together and Automation, Sensor, Template and Script go together but I don't think that is the case.

In my home assistant set up I have the following includes:

group: !include groups.yaml
light: !include lights.yaml
automation: !include_dir_merge_list automations
script: !include scripts.yaml
scene: !include scenes.yaml

In my main configuration.yaml I put sensors and switches as well as more global config. Given this set up can you tell me where each of the sections of yaml goes?

tanc commented 3 years ago

I've just had a breakthrough ;-) It turns out I was confusing myself by overwriting the top level sensor yaml key further down my configuration.yaml file. So instead of having one sensor key I had two, the second one overriding the first, likewise with the switch key. I can now see both the switch and the sensor and can control the power level with a slider, which I can confirm works with my Palazzetti Ginger Idra 15 stove.

I will try to provide a PR with some clarification to the docs, once I'm sure I've got everything set up.

tanc commented 3 years ago

The only thing I'm confused about now is the purpose of input_text.text_test which is used in Check pwr state automation. I got it working by adding an input_text key like this in my configuration.yaml:

input_text:
  text_test:
    name: Power

This appears to be only for visual output of the power level in a card and not used elsewhere. Can you confirm its purpose?

One change I made which might be useful is to the stove_fire_lvl. Due to the initial_value being set to 1 on the stove_fire_lvl, when restarting home assistant the wrong power level in my slider was being displayed, always 1. So I updated my Check pwr state automation to also set that value:

- id: '1'
  alias: Check pwr state
  trigger:
    platform: state
    entity_id: palazzetti.stove
  action:
  - service: input_text.set_value    
    data_template:
      entity_id: input_text.text_test # don't miss to create a input_text "text_test" for test this script
      value: "{{ state_attr('palazzetti.stove', 'PWR') }}"
  - service: input_number.set_value
    data:
      entity_id: input_number.stove_fire_lvl
      value: "{{ states('palazzetti.pwr') }}"
tanc commented 3 years ago

I see com error in my logs, although it doesn't seem to affect anything. From the code it looks like when there is no response from the cbox:

image

In my logs (set to debug as per instructions) I see:

2021-02-06 12:03:57 DEBUG (SyncWorker_6) [custom_components.palazzetti] request stove SET POWR
2021-02-06 12:04:44 DEBUG (SyncWorker_7) [custom_components.palazzetti] request stove SET POWR
2021-02-06 12:05:29 DEBUG (SyncWorker_7) [custom_components.palazzetti] request stove SET POWR
2021-02-06 12:05:35 DEBUG (SyncWorker_4) [custom_components.palazzetti] retry for op :SET POWR avoided
2021-02-06 12:18:16 ERROR (MainThread) [custom_components.palazzetti] Error returned by CBox
2021-02-06 12:23:46 ERROR (MainThread) [custom_components.palazzetti] Error returned by CBox
2021-02-06 12:24:47 ERROR (MainThread) [custom_components.palazzetti] Error returned by CBox
2021-02-06 12:27:47 ERROR (MainThread) [custom_components.palazzetti] Error returned by CBox
2021-02-06 12:29:46 ERROR (MainThread) [custom_components.palazzetti] Error returned by CBox

Are the errors anything to worry about?

tanc commented 3 years ago

I'm now seeing these errors:

2021-02-06 12:59:16 DEBUG (MainThread) [custom_components.palazzetti] get_request() response false for op GET ALLS
2021-02-06 12:59:23 ERROR (MainThread) [custom_components.palazzetti] Error during api request : http status returned is 502
2021-02-06 12:59:23 DEBUG (MainThread) [custom_components.palazzetti] get_request() response false for op GET ALLS
2021-02-06 12:59:38 ERROR (MainThread) [custom_components.palazzetti] Error during api request : http status returned is 502
2021-02-06 12:59:38 DEBUG (MainThread) [custom_components.palazzetti] get_request() response false for op GET ALLS
tanc commented 3 years ago

Weirdly the CBox crashed and stopped responding:

2021-02-06 13:19:59 DEBUG (MainThread) [custom_components.palazzetti] get_request() response false for op GET ALLS
2021-02-06 13:20:29 ERROR (MainThread) [custom_components.palazzetti] Error during api request: Cannot connect to host 192.168.1.16:80 ssl:default [Connect call failed ('192.168.1.16', 80)]

nmap showed the device and port were up but it wasn't happy. After a reboot of the CBox all is back to normal.

Vost3 commented 2 years ago

Hello.

Sry for the delay. The issue still appear ? Did you connect on ethernet or wifi at your cbox ? The wifi of the cbox seem to be poor.

FilipNaudts commented 2 years ago

Hi all, I wanted to add pallazzetti but I get the following error: Component error: palazzetti - Integration 'palazzetti' not found. I don't understand why. Any idea?

tanc commented 2 years ago

@Vost3 no problem, I ended up using an ethernet cable and I was able to successfully use the integration for the rest of the season. As the weather is getting cold again I've just got things back up and running again. The integration no longer worked due to home assistant changes so I've issued a PR #9 with the fixes.