slimcdk / esphome-custom-components

Components for ESPHome
12 stars 3 forks source link

Fix example config motor_load() #3

Closed morotsgurka closed 2 months ago

morotsgurka commented 2 months ago

Seem like motor_load() was changed to get_motor_load() in commit #a893b85 Not sure if more references need changing in the actual code for example when using:

sensor:
  - platform: tmc2209
    type: motor_load
    name: Motor load
    update_interval: 100ms

But i did not get any compile errors from using it?

slimcdk commented 2 months ago

Good catch! I missed that when renaming the method. I do however think that the template sensor example should be replaced entirely with the tmc2209 sensor instead at some point. I'll merge this and fix the docs later this week. Unless you have the time to do more changes? Thank you!

morotsgurka commented 2 months ago

No problem!

Getting multiple drivers working over the same UART would be very nice to be able to daisy-chain them together. Also, adding a simple wiring diagram would probably help a lot of people starting out!

slimcdk commented 2 months ago

In theory that would work as addressing is already taking into account. However, ESPHome doesn't allow multiple components to utilize the same UART.

Diagrams for the IC and most common breakout boards will come once I have a few hours to draw some pretty ones using fritzing or similar. Hopefully soon. :grin: