remove changing values directly and only deactivating params after calibration...

Changing values in the ui when clicking e.g. fixCenter is incorrect.
The user needs to recalibrate to bring his changes into effect

Closes #474 (closed)

ToDo

• user needs to be made aware he has to recalibrate

Reviewer Checklist

Formatting

• the pre-build checks succeed

General code quality

• naming conventions are met (see .clang-tidy for detailed information)
• no static analyzer warnings in new code parts (e.g., use clang-tidy for checking)

General usability

• old versions of pet-files are still loadable

Only if changes in UI

• new elements are also saved and loaded from pet-file
• check if tab order is still correct
• all new SpinBoxes are promoted
• new keybindings added to Petrack::keyBindings()

changed milestone to %1.0.0 - old

added GUI calibration labels

assigned to @l.dressen

changed the description

added 2 commits

• 247e944e - review comments + increment petrack version
• 8a90589d - change last path for autocalib instead of actually adding a calib file

Compare with previous version

marked the checklist item user needs to be made aware he has to recalibrate as completed

added 1 commit

• 1880a3ca - grey out params immediately

Compare with previous version

marked this merge request as ready

marked the checklist item the pre-build checks succeed as completed

marked the checklist item naming conventions are met (see .clang-tidy for detailed information) as completed

marked the checklist item no static analyzer warnings in new code parts (e.g., use clang-tidy for checking) as completed

marked the checklist item old versions of pet-files are still loadable as completed

We already discussed this, but to put it into writing:

The .pet-file should always contain the checkbox-states with which the project was calibrated. In the case that the user selects "continue with current calibration", this is not the same as the currently selected checkboxes.

added 1 commit

• b7661715 - review comments + calibSettings struct

Compare with previous version

added 1 commit

• 42e01a4a - setCalibsettings after successful calibration, added testcase for changing...

Compare with previous version

The only thing left is kind of a nitpick. Please resolve it yourself after you addressed it and merge this; I'll already approve this MR

approved this merge request

added 1 commit

• dbc76e0a - add comment

Compare with previous version

resolved all threads

added 5 commits

• dbc76e0a...8dade8b9 - 4 commits from branch master
• 6a3c10e0 - Merge remote-tracking branch 'origin/master' into...

Compare with previous version

enabled an automatic merge when the pipeline for 6a3c10e0 succeeds

merged

mentioned in commit cd9bbc08