search

arthurrump / esphome-opentherm

Create your own smart modulating thermostat using the OpenTherm component for ESPHome
BSD 2-Clause "Simplified" License
69 stars 37 forks source link

please help #25

Open krzysiek200 opened 8 months ago

krzysiek200 commented 8 months ago

Validation OK but...

INFO ESPHome 2023.10.3 INFO Reading configuration /config/esphome/thermostat-pid-complete.yaml... INFO Generating C++ source... Traceback (most recent call last): File "/usr/local/bin/esphome", line 33, in <module> sys.exit(load_entry_point('esphome', 'console_scripts', 'esphome')()) File "/esphome/esphome/__main__.py", line 1036, in main return run_esphome(sys.argv) File "/esphome/esphome/__main__.py", line 1023, in run_esphome rc = POST_CONFIG_ACTIONS[args.command](args, config) File "/esphome/esphome/__main__.py", line 403, in command_compile exit_code = write_cpp(config) File "/esphome/esphome/__main__.py", line 189, in write_cpp generate_cpp_contents(config) File "/esphome/esphome/__main__.py", line 201, in generate_cpp_contents CORE.flush_tasks() File "/esphome/esphome/core/__init__.py", line 650, in flush_tasks self.event_loop.flush_tasks() File "/esphome/esphome/coroutine.py", line 246, in flush_tasks next(task.iterator) File "/esphome/esphome/__main__.py", line 181, in wrapped await coro(conf) File "/data/external_components/d4d3a138/components/opentherm/number.py", line 34, in to_code keys = await generate.component_to_code( File "/data/external_components/d4d3a138/components/opentherm/generate.py", line 107, in component_to_code entity = await create(conf, key, hub) File "/data/external_components/d4d3a138/components/opentherm/number.py", line 18, in new_openthermnumber await number.register_number(var, config, min_value = config[input.CONF_min_value], max_value = config[input.CONF_max_value]) TypeError: register_number() missing 1 required keyword-only argument: 'step'

en-ver commented 8 months ago

@krzysiek200 your esphome config would help

crazypaddo commented 8 months ago

Hi i have the same error using the example from git, it is something with number config when i remove that part it works compiling.

pewu78 commented 8 months ago

See this https://github.com/arthurrump/esphome-opentherm/issues/17#issuecomment-1719748608 for solution. PR https://github.com/arthurrump/esphome-opentherm/pull/15 should really be merged...