roomtemperature interrupt

[robert-b]

with the latest heishamon firmware 4.1.3 i see a strange issue.

the roomtemperature shown on the lcd is 22 degree celsius.
this is a OT EMOS thermostat and i assume it just works a the probem has only be shown with firmware versions 4.1 and above.

the issue has been popped up more then once.
it might be race condition.

[robert-b]

on ?roomTemp then
if isset(@Defrosting_State) && @Defrosting_State == 0 then
if (?roomTemp > 0.1) then
#setPointRoomTemperature = ?roomTempSet;
#RoomTemperature = ?roomTemp;

        if isset(@Powerful_Mode_Time) && @Powerful_Mode_Time == 0 then
            if #RoomTemperature >= (#setPointRoomTemperature + #pvOffset) then
                stopHeatPump();
            elseif #RoomTemperature < #levelBoostTemperature then
                startHeatPump();
            end
        end
    end
end

end
the only routine where the temperature has been set on the esp32 board.

[IgorYbema]

Maybe explain your issue? It is hard to understand only from the graphs

[IgorYbema]

Also understand that opentherm doesn't have a great error checking buildin (only a parity bit check). False reads can occur so you might need to filter out sudden changes in values yourself

[robert-b]

Reliable Firmware would implement a sleep rate limiter.Am 05.05.2026 11:03 schrieb IgorYbema ***@***.***>:IgorYbema left a comment (heishamon/HeishaMon#880) Also understand that opentherm doesn't have a great error checking buildin. False reads can occur so you might need to filter out sudden changes in values yourself —Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID: ***@***.***>

[robert-b]

A value of zero comes through as ?roomtempAm 05.05.2026 11:00 schrieb IgorYbema ***@***.***>:IgorYbema left a comment (heishamon/HeishaMon#880) Maybe explain your issue? It is hard to understand only from the graphs —Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID: ***@***.***>

[IgorYbema]

Sorry but I don't understand this way of issue reporting

[robert-b]

I read 22 degree Celsius on the open therm LCDThe variable ?roomtemp says zero degree Celsius.Opentherm::process is unreadableRefactor it introducing the old open therm::processresponseThe variable declaration opentherm ot();Is wrongIt should be volatile opentherm ot();The variable ot is used within the interrupt context I hope this helpsAm 05.05.2026 13:29 schrieb IgorYbema ***@***.***>:IgorYbema left a comment (heishamon/HeishaMon#880) Sorry but I don't understand this way of issue reporting —Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID: ***@***.***>

[robert-b]

The opentherm class does nothing know about the amount of bits transfered.The spec says that the data is transmitted in 32 bits.Add a bits received counter and you know much more.Invalid frames can be handled by adding this bit counter.Just saying the opentherm data transfer is weak is easy.Am 05.05.2026 12:52 schrieb Robert Bouwens ***@***.***>:Reliable Firmware would implement a sleep rate limiter.Am 05.05.2026 11:03 schrieb IgorYbema ***@***.***>:IgorYbema left a comment (heishamon/HeishaMon#880) Also understand that opentherm doesn't have a great error checking buildin. False reads can occur so you might need to filter out sudden changes in values yourself —Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID: ***@***.***>

[IgorYbema]

Sorry but your posts are unreadable

[IgorYbema]

And the 32 count is there already:
if (responseBitIndex < 32) {

[robert-b]

CorrectAm 05.05.2026 17:24 schrieb IgorYbema ***@***.***>:IgorYbema left a comment (heishamon/HeishaMon#880) And the 32 count is there already: if (responseBitIndex < 32) { —Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID: ***@***.***>