Unify INI values

[hansu]

Fixes #3238 and #2335.

To be done

• Adapt Gmoccapy - change spindle => spindle_0 in *.ini files #3488
• Update sample configs for DEFAULT_LINEAR_VELOCITY [TRAJ] -> [DISPLAY]
• Increase INI file version and set up update-ini script

[zz912]

* [ ]  Adapt [Gmoccapy - change spindle => spindle_0 in *.ini files #3488](https://github.com/LinuxCNC/linuxcnc/pull/3488)

Feel free remake (or close) #3488 how you need. Git is not my friend, so I cant help you.

#3488 also does:

• INI file from PNCconf is more verbose in build_INI.py
• correct MACHINE in 7_axis.ini
• correct MACHINE in 6_axis.ini

[andypugh]

update_ini is designed to be scriptable, (-d and -f flags) and I think I had a meta-script that ran it on all the sample configs.
I probably can't find it any more, but I could have a look.

[hansu]

update_ini is designed to be scriptable, (-d and -f flags) and I think I had a meta-script that ran it on all the sample configs. I probably can't find it any more, but I could have a look.

Yes that would be helpful. I only used tested with the -d option so far.
The renaming stuff is done so far, only some parts which moves or copy some values are missing.
And a bit refactoring maybe.
Also a little bug where it generates two [KINS] sections...

[andypugh]

Yes that would be helpful.

Sorry, I looked and it's nowhere obvious. It might be sat in a git stash somewhere, but I think it's probably been lost.

I doubt it was anything special, I am not much of a scripting wizard.

[hansu]

I copied now

• [TRAJ]DEFAULT_LINEAR_VELOCITY -> [DISPLAY]DEFAULT_LINEAR_VELOCITY
• [TRAJ]MIN_LINEAR_VELOCITY -> [DISPLAY]MIN_LINEAR_VELOCITY

The values are/will not be not used in TRAJ any more. Do you think we should delete them from TRAJ or keep them as a precaution ?

[c-morley]

Ate you sure they ate not used in the trajectory planner?
Iirc they are used in non trivial machines.

[hansu]

Ate you sure they ate not used in the trajectory planner? Iirc they are used in non trivial machines.

DEFAULT_LINEAR_VELOCITY I overlooked, but MIN_LINEAR_VELOCITY is couldn't find in the trajectory planner.

[c-morley]

Ahh yes DEFAULT is what I was recalling.