feat(dataclass): add a method to split points by sensors

[ktro2828]

What

This pull request adds a new method for splitting point clouds by sensor and introduces comprehensive unit tests for the PointCloud dataclasses. The most significant changes are the addition of the split_by_sensor method and a new test suite that covers point cloud operations, validation, and file I/O.

PointCloud API Enhancements

• Added a split_by_sensor method to the PointCloud class, allowing a point cloud to be split into multiple point clouds based on sensor tokens using the metainfo attribute. This method returns a dictionary mapping sensor tokens to their corresponding PointCloud instances and raises an error if metainfo is missing.

Testing Improvements

• Added a new test suite in tests/dataclass/test_pointcloud.py that covers:
  • Construction and validation of Stamp, PointCloudSourceInfo, and PointCloudMetainfo objects.
  • Point cloud transformation operations (translate, rotate, transform).
  • Validation of source coverage and error handling for invalid configurations.
  • The new split_by_sensor method, including error cases and ensuring independence of split point clouds.
  • File reading for LidarPointCloud and SegmentationPointCloud, including points, labels, and metainfo parsing.

[github-actions]

☂️ Python Coverage

current status: ❌

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
4236	3558	84%	50%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
t4_devkit/dataclass/pointcloud.py	71%	🔴
TOTAL	71%	🔴

updated for commit: 35ace04 by action🐍

[Copilot]

Pull request overview

This PR extends the PointCloud dataclass API with a split_by_sensor() helper that partitions a concatenated point cloud into per-sensor point clouds using metainfo.sources, and adds a new unit test suite covering point cloud validation, transforms, splitting, and file I/O.

Changes:

• Added PointCloud.split_by_sensor() to slice point buffers into per-sensor PointCloud instances based on metainfo.sources.
• Added a comprehensive tests/dataclass/test_pointcloud.py suite covering timestamp/metainfo parsing, metainfo coverage validation, transform ops, splitting, and from_file readers.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
t4_devkit/dataclass/pointcloud.py	Adds split_by_sensor() to the PointCloud base class.
tests/dataclass/test_pointcloud.py	Introduces unit tests for PointCloud-related dataclasses and I/O, including the new split method.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.