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.

Sign up for GitHub

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

Jump to bottom

Lanedet dataset curation tusimple #35

Merged
merged 7 commits into from
Jan 8, 2025
Merged

Lanedet dataset curation tusimple #35

merged 7 commits into from
Jan 8, 2025

Conversation

devang-marvania
Copy link
Contributor

Hi @m-zain-khawaja ,

Submitting this PR for issue#22: #22 (comment)

@devang-marvania
Copy link
Contributor Author

devang-marvania commented Jan 6, 2025 •
edited
Loading

Hi @m-zain-khawaja ,

Please see the answers to your questions regarding this pull request below.

  1. Example of visualization and binary image:
    Raw image:
    00002
    Visualization image:
    00002
    Binary image:
    00002

  2. How many total images: 6394 each in raw image, visualization and binary image folder. Total: 6394 x 3 = 19,182 images.

  3. Are all images PNG?: Yes

  4. Snap shot of JSON file:
    image
    image
    image

@devang-marvania
Copy link
Contributor Author

devang-marvania commented Jan 7, 2025 •
edited
Loading

Dataset uploaded on Kaggle: Link

m-zain-khawaja
m-zain-khawaja requested changes Jan 7, 2025
View reviewed changes
Copy link
Collaborator

@m-zain-khawaja m-zain-khawaja left a comment

Choose a reason for hiding this comment

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

@devang-marvania - thank you for submitting this PR. The main scripts for create_lane and create_path look good. Could you however please remove 'trial.py' from the create_path/TuSimple folder from the git tracking as well as the .vscode folders in create_lane/TuSimple and create_path/TuSimple - once you have made these updates this PR is good to be merged.

@devang-marvania
Copy link
Contributor Author

HI Zain, the requested files are removed now. Thanks!

m-zain-khawaja
m-zain-khawaja approved these changes Jan 8, 2025
View reviewed changes
Copy link
Collaborator

@m-zain-khawaja m-zain-khawaja left a comment

Choose a reason for hiding this comment

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

Thanks, these look good

@m-zain-khawaja m-zain-khawaja merged commit 6baa854 into autowarefoundation:lanedet-dataset-curation-tusimple Jan 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-zain-khawaja m-zain-khawaja m-zain-khawaja approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devang-marvania @m-zain-khawaja