maybe this can be useful : https://forum.mysensors.org/topic/6948/synchronising-light-switch
I use it with version 1requestState or 2 requestTime() on some sketches
(I modified the structure, moved some parts to functions to make it more readable )
@virtualmkr said in CAN bus transport implementation:
I'm not a "radio guy", but I think this code change looks fine. When sanity check fails then MySensors core calls transportInit() which would try to re-initialise the MCP2515 module.
Ok. I'll update code.
@virtualmkr said in CAN bus transport implementation:
I have done some modifications in the CAN transport files.
@Adam-Slowik I would like to create a PR in your forked repo. Then we can discuss the code changes in GitHub. Would that be Ok for you - do you have time for this?
Sure. Make PR. I'll try to review it within 48h.
@virtualmkr said in CAN bus transport implementation:
When I send 32 byte message I observed the sub telegrams do not always arrive in send order
Other limitation I see is the limited CAN telegram buffer of 3 packets in the MCP2515 module. I'm afraid for reliable operation with MySensors a change from polling to interrupt mode is required.
I think this two issues are linked. Now packets are sent in correct order (could You verify it?). If packets are sent in short periods We should use interrupt to keep MCP2515 buffers empty. When We fail to keep buffers empty (due to polling mechanism) We receive sub telegrams in mixed order. This theory could be true if MCP2515 buffers are implemented as LIFO. I'll look into documentation to figure it out.
Anyway both fixes are worth to implement. I'll try to do this over weekend.
@ncollins Impressive! Looks as though you even modeled the exact shape of the coincell holder. And you apparently designed the 3D printer snap enclosures as well. Even just the 3D printed enclosure modelling is quite a task all by itself. I hope to one day evolve into doing that too, but for now the Hammond enclosures are going to be my short-term solution. The smallest Hammond is still not as small as what you've managed to produce, but at 36mm wide for an off-the-shelf ABS enclosure, it's too wide by only about 10mm. Still.... one day I want to 3D print an enclosure to be as small as possible.
Great, thanks for your reply and help.
I was able to do what I wanted, I give you some code snippets to make several messages via a for loop,
I'm not a code proffesional but this works:
// Initialize messages
MyMessage msgInA(0,V_CURRENT);
MyMessage msgWhA(0,V_WATT);
#define NSENSORS 6 //used current sensors
void setup()
{
//initialize energy monitor CurrentSensor
for (int i=0; i<NSENSORS; i++)
{
wh[i] = { 0.0 }; //initialize wh
lwhtime[i] = {0}; //initialize time
present(i, S_MULTIMETER);
}
for (int i=0; i<NSENSORS; i++)
{
present(i, S_POWER);
}
}
void loop()
{
for (int i=0; i<NSENSORS; i++)
{
send(msgInA.setSensor(i).set(Irms[i], 1));
send(msgWhA.setSensor(i).set(wh[i], 1));
}
}
Thanks
@alowhum Sorry, I have abandoned this project. It didn't garner the interest I hoped it would, and I’ve moved on to other microcontrollers/radios/networks since then. This is hosted on GitHub’s free static hosting, so it should continue to work as long as they are in business, and it should work on devices as it was 5-6 years ago. But I won’t be supporting it and won’t continue to build on it. Glad to hear that you like it!
