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

🗺Added support for multiple coordinate systems #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rmeertens
Copy link

No description provided.

@@ -1,4 +1,7 @@
"""Converts dynamic object annotations to KITTI format."""
import os, sys
sys.path.append(os.path.join(os.path.dirname(__file__), "../"))

Choose a reason for hiding this comment

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

Why is this needed?

with open(join(target_path, f"{id_:06d}.txt"), "w") as target_file:
target_file.write("\n".join(target_anno))


def _parse_args():
parser = argparse.ArgumentParser(description="Convert annotations to KITTI format")
parser.add_argument("--dataset-dir", required=True, help="Root dataset directory")
parser.add_argument("--dataset-dir", required=True, help="Root dataset directory. Assumed to contain json files with label-coordinates in the zenseact lidar coordinate system")

Choose a reason for hiding this comment

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

Please keep the line width to 100 by adding a new line.

parser.add_argument("--target-dir", required=True, help="Output directory")
parser.add_argument("--target_coordinate_system", required=True, help="What coordinate system to convert files to")

Choose a reason for hiding this comment

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

Would be nice to add choices=['kitti_camera', 'zenseact_lidar']

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants