Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: extract IMU from GoPro videos #692

Merged
merged 7 commits into from
Dec 10, 2024
Merged

feat: extract IMU from GoPro videos #692

merged 7 commits into from
Dec 10, 2024

Conversation

ptpt
Copy link
Member

@ptpt ptpt commented Dec 6, 2024

This pull request introduces significant changes to the mapillary_tools/geotag/gpmf_parser.py file, focusing on enhancing telemetry data extraction and handling. The most important changes include the addition of new functions for matrix calibration and telemetry data extraction, as well as updates to existing functions to accommodate these enhancements.

Enhancements to telemetry data extraction:

  • mapillary_tools/geotag/gpmf_parser.py: Added new functions _is_matrix_calibration, _build_matrix, _apply_matrix, _flatten, _get_matrix, _scale_and_calibrate, _find_first_telemetry_stream, and extract_telemetry_data to support telemetry data extraction and processing. [1] [2]
  • mapillary_tools/geotag/gpmf_parser.py: Updated _extract_points_from_samples to extract and interpolate accelerometer, gyroscope, and magnetometer data in addition to GPS points.

Refactoring and code improvements:

Addition of new data structures:

  • mapillary_tools/imu.py: Introduced GyroscopeData, AccelerationData, and MagnetometerData classes to represent gyroscope, accelerometer, and magnetometer readings respectively.

Test updates:

  • tests/cli/gpmf_parser.py: Added _parse_samples function to parse and dump telemetry samples for testing purposes, and updated the main function to utilize this new function for sample dumping. [1] [2] [3] [4] [5]

@ptpt ptpt merged commit 8bcc73d into main Dec 10, 2024
16 checks passed
@ptpt ptpt deleted the feat-extract-imu branch December 10, 2024 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants