Skip to content

feat: add directory path as property#179

Merged
ktro2828 merged 1 commit into
mainfrom
feat/tier4/directory-path
Aug 28, 2025
Merged

feat: add directory path as property#179
ktro2828 merged 1 commit into
mainfrom
feat/tier4/directory-path

Conversation

@ktro2828

Copy link
Copy Markdown
Collaborator

What

This PR adds property of the path to directories included in T4 dataset, such as map and input_bag.

Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
Copilot AI review requested due to automatic review settings August 28, 2025 10:52
@github-actions github-actions Bot added the new-feature New feature or request label Aug 28, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the Tier4 class by adding a property for the ROS bag directory path and refactoring schema directory handling. The changes improve the API by providing convenient access to the input_bag directory and removing a hardcoded class variable.

  • Added bag_dir property to return the path to the input_bag directory
  • Removed hardcoded schema_dir class variable and updated references to use annotation_dir
  • Simplified print statement for loading T4 tables

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread t4_devkit/tier4.py
@github-actions

github-actions Bot commented Aug 28, 2025

Copy link
Copy Markdown
Contributor

☂️ Python Coverage

current status: ❌

Overall Coverage

Lines Covered Coverage Threshold Status
2653 1835 69% 50% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
t4_devkit/tier4.py 19% 🔴
TOTAL 19% 🔴

updated for commit: ea2c881 by action🐍

@ktro2828 ktro2828 merged commit bc64ee1 into main Aug 28, 2025
5 checks passed
@ktro2828 ktro2828 deleted the feat/tier4/directory-path branch August 28, 2025 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants