Skip to content

Fix old pet files without EXTENDED_MODEL parameter

d.kilic requested to merge 528-loading-calib into master

There exist old pet files from before the EXT_MODEL_ENABLED parameter existed which have used the new model. During our restructuring we subtly broken loading these; we always load them as the old model, even if the extended model was used. So one needed to activate the old model to then use the (correct) extended model...

This fixes loading such projects. (Examples can be found in CroMa, e.g. 3B071_blank_C_corrected_trimmed.pet)

Reviewer Checklist

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
  • documentation updated (if applicable)

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()

Closes #528 (closed)

Merge request reports

Loading