Typo regarding "G52", "G92" and DISABLE_G92_PERSISTENCE

[Unl0gic]

On the page linked here it says:

• 5210 - 1 if "G52" or "G92" offset is currently applied, 0 otherwise. Volatile by default; persistent if DISABLE_G92_PERSISTENCE = 1 in the [RS274NGC] section of the INI file.

• 5211-5219 - Shared "G52" and "G92" offset for X, Y, Z, A, B, C, U, V & W. Volatile by default; persistent if DISABLE_G92_PERSISTENCE = 1 in the [RS274NGC] section of the INI file.

This seems to have been inverted, shouldn't it say that the parameters in question are persisted by default and volatile only if "DISABLE_G92_PERSISTENCE = 1" is set?

[andypugh]

It probably should. But do you have the time to check what actually happens? Possibly the INI file entries have confusing names?

[Sigma1912]

related https://github.com/LinuxCNC/linuxcnc/pull/2916

[Sigma1912]

I created a pull request for this: https://github.com/LinuxCNC/linuxcnc/pull/2919

Thanks for reporting.

[hansu]

@Unl0gic Is this resolved now?

[Unl0gic]

@hansu Yes from what I can see this issue has now been corrected.