Skip to content
Snippets Groups Projects

Resolve "Crash after intrinsic calibration with old model"

Merged d.kilic requested to merge 367-crash-old-calibration into master

Crash was due to operator== of IntrinsicCameraParams trying to compare distCoeffs of different size.

I changed AutoCalib to always return the maximal size, padding smaller models with zeros. This should lead to identical behaviour (@l.dressen can you confirm?).

Additionally, a new regression test case for the old model has been added, using data from commit 76374105 (last commit before IntrinsicBox-MR)

Closes #367 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • 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
Please register or sign in to reply
Loading