This mainly affected setting the work coordinate with the mouse, which can result in numbers with lots of digits. This doesn't work with my particular controller (uCNC), but more generally always results in a very long command being shown in the UI, and is inconsistent with other code.
This does make some uses of _wcsSet look a bit odd:
some places pass '"0"' strings (or empty strings), presumably to avoid being formatted as "0.0", but this is redundant if they are always converted to a float first.
some places round numbers before calling this, but that's not necessary.
I can fix these if desired. It's also unclear what the 10000mm limit is for; looks like this was introduced in this commit d8dbd633b70a1ded4f291df86d44bd527cd1c755.
This mainly affected setting the work coordinate with the mouse, which can result in numbers with lots of digits. This doesn't work with my particular controller (uCNC), but more generally always results in a very long command being shown in the UI, and is inconsistent with other code.
This does make some uses of
_wcsSet
look a bit odd:I can fix these if desired. It's also unclear what the 10000mm limit is for; looks like this was introduced in this commit d8dbd633b70a1ded4f291df86d44bd527cd1c755.