IRMP-org/IRMP

SIEMENS protocol not detected

eku opened this issue · 12 comments

eku commented

Bug Report

Arduino Platform

  • Arduino ATmega328* board (UNO, Nano)
  • Arduino ATmega2560 board (Mega)
  • Arduino ATmega32U4 board (Leonardo)
  • Arduino megaAVR board (NanoEvery)
  • Arduino SAM board (Due)
  • Arduino SAMD board (Zero, MKR*)
  • ESP8266 board
  • ESP32 board
  • STM32 board with "STM32 Boards (selected from submenu)" core
  • STM32 board with "STM32F1 Boards (STM32duino.com)" core
  • Apollo3 board
  • Teensy board
  • Other - please specify

IDE

  • Arduino IDE
  • Arduino Pro IDE
  • Sloeber IDE
  • PlatformIO IDE
  • Other - please specify

IR-Protocol

  • Unknown
  • Sony
  • NEC
  • Samsung
  • RC5, RC6
  • Kaseikyo
  • Denon
  • JVC
  • Lego
  • Matsushita
  • Mitsubishi
  • Panasonic
  • Sanyo
  • Sharp
  • Telefunken
  • Siemens + Nokia

Example to reproduce the issue

  • AllProtocols
  • Callback
  • Interrupt
  • OneProtocol
  • ReceiveAndSend
  • ReceiveAndSendDynamicPins
  • RFProtocols
  • SimpleReceiver
  • SimpleSender
  • Other

Pin(s) used for IR-receive, if not default

I use dynamic PINs (GPIO5) with Tasmota configuration.

Version

  • Yes I use the latest release and verified this!
  • Other - please specify

Current behavior

I use a chinese IR TUYA device flashed with Tasmota and my port of IRMP to Tasmota.

For comparison I have a selfmade module on with ATmega32 and native IRMP (no Arduino). This one detects all protocols correctly.

IRMP is compiled with the following settings:

#define IRMP_PROTOCOL_NAMES 1
#define IRMP_IRSND_ALLOW_DYNAMIC_PINS 1
#define F_INTERRUPTS 20000 // Instead of default 15000 to support LEGO + RCMM protocols
#define USE_ONE_TIMER_FOR_IRMP_AND_IRSND 1
#define tone(a,b) void() // no tone() available on ESP32
#define noTone(a) void()
#define IRMP_SUPPORT_GRUNDIG_PROTOCOL           1
#define IRMP_SUPPORT_MATSUSHITA_PROTOCOL        1
#define IRMP_SUPPORT_NEC_PROTOCOL               1
#define IRMP_SUPPORT_NOKIA_PROTOCOL             1
#define IRMP_SUPPORT_NUBERT_PROTOCOL            1
#define IRMP_SUPPORT_ONKYO_PROTOCOL             1
#define IRMP_SUPPORT_RC5_PROTOCOL               1
#define IRMP_SUPPORT_SIEMENS_PROTOCOL           1
#define IRSND_SUPPORT_GRUNDIG_PROTOCOL          1
#define IRSND_SUPPORT_MATSUSHITA_PROTOCOL       1
#define IRSND_SUPPORT_NEC_PROTOCOL              1
#define IRSND_SUPPORT_NOKIA_PROTOCOL            1
#define IRSND_SUPPORT_NUBERT_PROTOCOL           1
#define IRSND_SUPPORT_ONKYO_PROTOCOL            1
#define IRSND_SUPPORT_RC5_PROTOCOL              1
#define IRSND_SUPPORT_SIEMENS_PROTOCOL          1

Expected behavior

Siemens and Nokia protocols are always correctly detected.

Error output

No serial output on Tasmota :(

The Siemens protocol is not detected at all.
The Nokia protocol is sometimes detected as SIEMENS.

Additional context

ArminJo commented

First it can be a problem of the IR receiver hardware.
Second, I do not know if Tasmota interferes with the polling of IRMP.
What about other protocols e.g. NEC?

eku commented

What about other protocols e.g. NEC?

No problem with other protocols, as far as tested.

I have reduced the sampling frequency to 15kHz as a test and Nokia is reliably detected. Siemens is at least sometimes detected, but not deterministically.

As receiver I use the same TSOP in both circuits.

The only thing that remains, as you already suspect, is the different implementation of the timers. Regarding the Arduino implementation for ESP, is there anything else you can think of where you can pull the lever and what else you could try?

ArminJo commented

How can I reproduce it? I have ESP's but only worked once with platformio and never with tasmota.
Are there some tutorials, or can I copy your platformio project.
Maybe some debugs can be enabled in IRMP to get an idea why it fails for Siemens.

eku commented

What is the next step with this topic? Has @ukw100 made an bug fix?

ArminJo commented

I commited the timing corrections.

eku commented

Version 3.4.0 does not improve the recognition.

It looks to me like the name does not match anything in the protocols. Ruwido is issued for Matshusita and Siemens for Nokia and so on.

Is the following code right?

(const char *)pgm_read_dword(&irmp_protocol_names[irmp_data.protocol])
  • 😕1
eku commented

Version 3.5.1 doesn't decode SIEMENS at all.

ArminJo commented

Can you check the current 3.6.0 version (not yet released). I changed the Siemens timings accordingly to https://github.com/j1rie/IRMP.
I always had false Simens recognitions for NEC, which was improved with the new timings.
How do you test it? do you have a real Siemens remote?
Can you post this in the microcontroller forum?

eku commented

I did upgrade to 1a62365, but no luck.

I don't know if it is necessarily due to the timing. I also use IRMP on an ATmega, but not the ARDUNIO version, but integrated in my own software and SIEMENS, RUWIDO and NOKIA are recognised. Could the ESP8266 cause more problems here than the AVR?

The recognition rate is rather worse at 3.60.

I have both a real SIEMENS remote and an universal remote that learned the codes from it. UKW originally implemented the SIEMENS protocol first based on my recorded scans. I use the same nick on mikrocontroller.net and I'm familiar with IRMP since years.

ArminJo commented

Until the last change I used Franks IRMP timing. And in the forum, he recommends the new timings of https://github.com/j1rie/IRMP. And of course I did not touch the logic!
So it must be the ESP8266 hardware (or timer) which makes the difference.