[3.3-rc1] Sensor issue with M308

Jackal

I have upgraded my Duet 2 Wifi from 3.3-beta2 (or maybe another 3.3-beta) to 3.3-rc1.
Both my bed and hotend sensors are thermocouple and after the upgrade both seems to have failed to work.
I am getting "string too long" error when using this command the config my thermocouple: M308 S0 P"spi.cs3" Y"thermocouple-max31855"
I saw the bug-fix "Thermocouple sensors could not be configured because M308 didn't accept more than 20 characters in the sensor type name" in the changelog. I would like to know if it is really fixed or I have done something wrong with the command which works fine before?

These are my sensor config:
; Bed sensor and heater
M308 S0 P"spi.cs3" Y"thermocouple-max31855"
M950 H0 C"bedheat" T0
M140 H0
;M307 H0 R1.319 C80.7 D3.06 S1.00 V24.2
M307 H0 R1 B1

; Hotend sensor and heater
M308 S1 P"spi.cs2" Y"thermocouple-max31855"
M950 H1 C"e1heat" T1
M307 H1 R2.643 C168.0 D3.48 S1.00 V24.1

dc42

@jackal please try the new 3.3RC1+1 firmware binary at https://www.dropbox.com/sh/dlb58vkmu1u4fkx/AAAelkXSfRKVwI6_yqRnhGHPa?dl=0.

dc42

@jackal please run M115 to check that you really are running firmware 3.3RC1.

Jackal

@dc42 said in [3.3-rc1] Sensor issue with M308:

FIRMWARE_NAME: RepRapFirmware for Duet 2 WiFi/Ethernet FIRMWARE_VERSION: 3.3RC1 ELECTRONICS: Duet WiFi 1.0 or 1.01 FIRMWARE_DATE: 2021-05-01 09:58:08

dc42

@jackal thanks. I confirm that this bug is still present in 3.3RC1.

Jackal

@dc42 Thanks for the quick follow up.

dc42

@jackal please try the new 3.3RC1+1 firmware binary at https://www.dropbox.com/sh/dlb58vkmu1u4fkx/AAAelkXSfRKVwI6_yqRnhGHPa?dl=0.

Jackal

@dc42 It's now working with 3.3RC1+1. Thank you.