Feature/io acthor heater element#64
Conversation
AC Thor is a 3 phase heating element, with a maximum power of 1-3kW per phase. This implementation reads from 3 digital inputs and sets the Power channel appropriately. Co-authored-by: Christian Lehne <51822163+clehne@users.noreply.github.com>
|
|
| } | ||
|
|
||
| private Integer calculatePower() { | ||
| Optional<Boolean> out1 = this.getOut1WriteChannel().getNextWriteValue(); // TODO Use getNextWriteValueAndReset |
There was a problem hiding this comment.
Maybe use var for more modern coding style
|
Nur eine Frage meinerseits - so wie ich die Impl verstehe ist es so, dass der AC-Thor in deiner Version nicht von einem Controller gesteuert werden kann, oder? Ich arbeite jetzt dann bald auch an einer anderen Implementierung eines Heizstabes (Smart mit Modbus) und Stefan meinte wir sollten das eventuell Abstahieren: https://community.openems.io/t/steuerbarer-heizstab-fur-pv-uberschuss/2525/16?u=sn0w3y Grüße ! |
Hallo @Sn0w3y, doch diese Implementierung kann direkt mit OpenEMS Bordmitteln genutzt werden. Habe dazu hier ein paar Zeilen geschrieben. Danke für den Hinweis auf die beiden Community Posts. |
Alles klar, danke für den Hinweis ! :) Aber es fehlt ja immer noch die PWM Steuerung, für die der Heizstab ja eigentlich auch benutzt werden sollte/kann, oder ? |
Instead of using the "if" cases and optionalBooleans when getting the nextwrite value -> use orElse false directly and use ternary operator also i called nextWriteValueAndReset (was a TODO)
|
|
3 participants
AC Thor is a 3 phase heating element, with a maximum power of 1-3kW per phase.
This implementation reads from 3 digital inputs and sets the Power channel appropriately.
@clehne wenn du einmal kurz bevor ich das ins öffentliche Repo als PR stelle drüber gucken könntest wäre das super, ich habe noch 2 änderungen in der Nature vorgenommen (line break bei persistence prio) und ich hab einen basic test implementiert.
der PR hat den aktuellen develop branch daher sind noch "mehr" änderungen als nur der acthor drin im PR