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: add IMU data in CAMM as experimental feature #694

Merged
merged 14 commits into from
Jan 17, 2025
Merged

Conversation

ptpt
Copy link
Member

@ptpt ptpt commented Dec 16, 2024

This pull request includes significant changes to the mapillary_tools package, primarily focused on enhancing the handling and processing of telemetry data. The most important changes are the introduction of new telemetry data types, refactoring of existing functions to support these new types, and updates to various modules to integrate the new telemetry data handling.

Telemetry Data Handling Enhancements:

  • mapillary_tools/camm/camm_builder.py: Refactored to support new telemetry data types (AccelerationData, GyroscopeData, MagnetometerData) and updated functions to handle these types. This includes the addition of the TelemetryMeasurement union type and the _multiplex function for combining different telemetry measurements. [1] [2] [3] [4] [5]

  • mapillary_tools/camm/camm_parser.py: Added parsing support for new telemetry data types and updated the extract_points and filter_points_by_elst functions to handle these types. Introduced the extract_telemetry_data function to extract all telemetry data from a CAMM video. [1] [2] [3] [4] [5]

Integration of Telemetry Data Types:

  • mapillary_tools/geotag/gpmf_parser.py: Updated to replace the old IMU data types with the new telemetry data types. This includes changes in the TelemetryData class and the _extract_points_from_samples function. [1] [2] [3] [4] [5] [6]

  • mapillary_tools/telemetry.py: Introduced new telemetry data classes (TelemetryMeasurement, AccelerationData, GyroscopeData, MagnetometerData) to replace the old IMU data classes.

@ptpt ptpt merged commit 37930a6 into main Jan 17, 2025
16 checks passed
@ptpt ptpt deleted the feat-write-imu branch January 17, 2025 08:33
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