Resolve "Ease usage of c3d visualization for time synchronization"
This MR adds a new dialog to edit the time offset (and in future position and rotation) of motion capturing data.
The dialog is accessible over File > Edit MoCap Settings
(the Time
is there for distinguishement to the future Position
and Rotation
)
As requested in the issue, the dialog is usable alongside the main window and changes are immeadiately shown.
Also, the option to select mocap persons to show was added (the show
checkbox). This option is saved in the pet file.
The time offset part of the mocap file dialog was removed since the functionality is now available through the new dialog.
Closes #391 (closed)
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()
Merge request reports
Activity
added high label
added 1 commit
- 76985e86 - fix layouting issues by manually setting minimum size of scrollAreas
added 15 commits
-
76985e86...418bc906 - 2 commits from branch
master
- d1399621 - WIP: Add functionality to dialog
- b83bbe2b - Add new files to repo
- 09846253 - Finish functionality
- 4c1c746c - Layouting and style
- 5fffd9dc - Save visibility of MoCapPerson in pet-file
- 79a53769 - Delete time offset from MoCap file dialog
- fa33c0a5 - Make dialog non-modal
- b96f82a6 - Select all and deselect all buttons
- e7bdbcad - Add license headers
- f499c7b9 - Remove superflous include
- 60191c2f - Formatting
- 1fcb1b9e - More formatting
- 74a15e09 - fix layouting issues by manually setting minimum size of scrollAreas
Toggle commit list-
76985e86...418bc906 - 2 commits from branch
enabled an automatic merge when the pipeline for 74a15e09 succeeds
mentioned in commit ca4c74a2