Fixed error with Tracking calibration loading. #73
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed issue in read_calib_file to accommodate for the difference in formatting in the tracking calibration files. The Tr fields don't have the : to separate the label from the calibration details, added a split to separate by the first space in this case
Previous calib_filepath depended on a variable "sequence_path", which was not defined. Set up the calib_filepath to the correct formatting for the path of the calibration file.
The Transform matrices in the calib file for the tracking dataset use the notation "Tr_cam_velo", instead of just "Tr". Updated the call with the correct transform name.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed error with the calibration file read that was generating a ValueError when trying to read from the tracking calibration file. The file is formatted differently from the from the Raw and Odometry calibration files, using spaces to separate the transform matrices from the values instead of a colon. Also updated the code in the tracking class to accommodate the differences in the calibration file. The _load_calibration file had a call to "self.sequence_path" that did not exist, updated the call with the correct formatting for the calibration file path.