Macro Parameter Syntax - Extra characters in param

adammhaile

I'm trying to make some custom M/G codes using macro files in /sys but am having some weird issues.
The one I started with is trying to make a command that is supposed to send serial data to an external device, via M118 a little simpler. But I cannot seem to even get the basics working on the time.
For the sake of testing I reduced the entired M800.g file I made to be nothing more than:

M118 P0 S{param.F}

So instead of sending to serial UART it will send to the control server log which I can view with journalctl on the SBC.

The problem is that, for some text, I'm getting extra characters and while consistent for specific inputs, I cannot find a pattern as to why.

For example, if I call the above with:

M800 F"test"

I see test output in the control server log.

But if I call:

M800 F"complete"

The control server shows:

complete{

Note the added curly brace.
It also shows the same if in the macro I do:

echo "{param.F}"

But what is causing that curly brace to get in there?
Any insight appreciated - I feel like it's probably a bug?

Please see version details below - all are latest from running apt-get upgrade:

Board: Duet 3 MB6HC (MB6HC)
DSF Version: 3.3.0
Firmware: RepRapFirmware for Duet 3 MB6HC 3.3 (2021-06-15)

adammhaile

@dc42 any thoughts? I've done some further testing and this is happening any time I try to use string params. Numeric is fine, but strings will just add the brace depending on the content.

chrishamm

@adammhaile Please try upgrading to the latest 3.4-rc1 version or wait a few more hours before we release 3.4-rc2. It should be fixed in that build.

adammhaile

@chrishamm Ah - glad to confirm it's not just me.
Though was hoping not to run a beta or RC - granted, how does one actually upgrade an SBC Duet 3 system to a beta or RC build? I spent 30 minutes trying to find that info the other day and all I could ever find was sudo apt-get update|upgrade but that's only release versions.
Or do I have to just do an old style manual firmware upgrade? I thought that for SBC you had to update everything together.

chrishamm

@adammhaile See here: https://docs.duet3d.com/User_manual/Machine_configuration/DSF_RPi

adammhaile

@chrishamm
Oh man - new site?! That looks awesome.
Thanks! I'll give it a shot.

adammhaile

@chrishamm Just got it upgraded and it works now, thanks!