Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
PeTrack
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Container Registry
Model registry
Analyze
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Terms and privacy
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Pedestrian Dynamics Empiricism
PeTrack
Merge requests
!261
fix: read reprojection error
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
fix: read reprojection error
364-reprojection-error-is-not-read-correctly-from-pet-file
into
master
Overview
7
Commits
5
Pipelines
6
Changes
11
Merged
d.kilic
requested to merge
364-reprojection-error-is-not-read-correctly-from-pet-file
into
master
1 year ago
Overview
7
Commits
5
Pipelines
6
Changes
11
Expand
read into
IntrinsicCameraParams
and then set all values at once
prevents invalidation of reprojection error because "intrinsics changed"
Closes
#364 (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()
Edited
1 year ago
by
Schrödter, Tobias
0
0
Merge request reports
Activity
All activity
Filter activity
Deselect all
Approvals
Assignees & reviewers
Comments (from bots)
Comments (from users)
Commits & branches
Edits
Labels
Lock status
Mentions
Merge request status
Tracking
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Please
register
or
sign in
to reply
Loading