@Trand Well, sorry, but nothing else comes to mind. If you have a scope or logic analyzer then you need to watch the actual signals on the wires and see what is happening.
If you don't, then you should pick up a cheap logic analyzer. You can get one for $10 and it's invaluable for tracking down things like this. I don't know where you are based, but I see them on Aliexpress all the time.
I've asked, because I cannot test it right now and thought maybe someone has done it before. In this case I will test it myself next week. Right now I have some code that compiles, so maybe it will also work
Referring to multimessage feature, I don't see any application yet (maybe I need to understand it better). More important to me is auto ID assignment for RS485 which seems to be also finally merged. Great!
At "https://www.mysensors.org/download/contributing#coding-guidelines-core-library", under "Signing CLA", the link to clahub.com is since long dead.
Maybe you can follow this example: https://github.com/chocolatey/choco/issues/2036
and replace with https://cla-assistant.io/ ?
JurjenBiewenga created this issue in chocolatey/choco
closed
CLAHub is down
#2036
So library gives us 3 interesting functions:
INT8U checkError(void);
INT8U errorCountRX(void);
INT8U errorCountTX(void);
Full description can be found in library source code and in documentation chapter 6:
https://ww1.microchip.com/downloads/en/DeviceDoc/MCP2515-Stand-Alone-CAN-Controller-with-SPI-20001801J.pdf
long story short:
INT8U checkError(void); returns:
-CAN_CTRLERROR when things are bad or very bad
-CAN_OK when things are ok or not so bad (error counters below 96)
I would like to chage implementation of sanity check to something like this:
bool transportSanityCheck(void)
{
return (CAN0.getError()==CAN_OK)
}
It won't blink diodes in case of communication problems, but will provide some feedback about communication to MySensors library.
2)INT8U errorCountRX(void); return error counter for received messages. AFAIK this counter is incremented when error occurs and decremented when message is received. I'm not a 'radio guy' but i think this could be returned as int16_t transportGetReceivingSNR(void) or int16_t transportGetReceivingRSSI(void). I know SNR and RSSI are radio related parameters, but perhaps We could bend the border here. Maybe some 'radio guy' could say something about this idea?
3)INT8U errorCountTX(void); same situation as with #2. 'Radio guy' opinion would by great.
Again ideas #2 and #3 are not solving problems, but providing feedback to MySensors.
Are My ideas ok, or do they do not pass sanityCheck()
