Release 0.12.0 #600
Merged
Release 0.12.0 #600
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
* Add support for adding custom fields to ``LidarScan`` s with ``add_field`` and ``del_field`` * Added per-request timeout arguments to ``SensorHttp`` * Added sensor user_data to ``sensor_info/SensorInfo`` and metadata files * Removed ``updated_metadata_string()`` and ``original_string()`` from ``sensor_info`` * Added ``to_json_string()`` to ``sensor_info`` to convert a ``sensor_info`` to a non-legacy metadata JSON string * Unified Python and C++ ``Packet`` and ``PacketFormat`` classes * Added ``validate`` function to ``LidarPacket`` and ``ImuPacket`` to check for ID and size mismatches * [BREAKING] LidarScan's width and height have been switched to size_t from ptrdiff_t in C++ * Refactor metadata parsing * Add ``get_version`` to ``sensor_info/SensorInfo`` to retrieve parsed version information * Add ``get_product_info`` to ``sensor_info/SensorInfo`` to retrieve parsed lidar model information * Raise an exception rather than throw an unrelated error when multiple viable metadata files are found for a given PCAP * Add ability to slice a scan source, returning a new sliced ScanSource * [BREAKING] Removed ``hostname`` in Python ``SensorInfo`` and ``name`` from C++ ``sensor_info`` * [BREAKING] Removed ``udp_port_lidar``, ``udp_port_imu`` and ``mode`` from C++ ``sensor_info`` * [BREAKING] Deprecated ``udp_port_lidar``, ``udp_port_imu`` and ``mode`` in Python ``SensorInfo``. These fields now point to the equivalent fields inside of ``SensorInfo::config``. * [BREAKING] Removed ``cols`` and ``frequency`` from ``LidarMode`` in Python * [BREAKING] Deprecated ```data``` and ``capture_timestamp`` from Python ``Packet`` * [BREAKING] Removed methods from Python ``ImuPacket`` and ``LidarPacket`` classes that simply wrapped ``PacketFormat`` * [BREAKING] Removed ``begin()`` and ``end()`` iterators of ``LidarScan`` in C++ * [BREAKING] Remove deprecated package stubs added in previous 0.11 release. * [BREAKING] Replaced integer backed ``ChanField`` enumerations with strings. * [BREAKING] Removed ``CUSTOM0`` through ``CUSTOM9`` ChanField enumerations. * [BREAKING] Extra fields in sensor metadata are now ignored and discarded if saved from the resulting ``sensor_info/SensorInfo`` * [BUGFIX] Prevent last scan from being emitted twice for PCAP * [BUGFIX] Fix corrupted packets due to poor handling of fragmented packet drop in PCAPs * [BUGFIX] Fix possible crash when working with custom UDPProfileLidars
twslankard
requested review from
Samahu,
celentes,
Krishtof-Korda,
kkaly,
chrisbayruns,
akatumalla-ouster and
matthew-lidar
akatumalla-ouster
approved these changes
Jul 3, 2024
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.
Description