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

Space character as the only delimiter when parsing yolo-format label files; this should be generalized to any whitespace character #1168

Closed
michael-mayo opened this issue Jun 13, 2023 · 3 comments
Assignees
Labels
🐛 Bug Something isn't working

Comments

@michael-mayo
Copy link

🚀 Feature Request

Space character as the only delimiter when parsing yolo-format label files; this should be generalized to any whitespace character.

Proposed Solution (Optional)

The issue is on line 231 of super_gradients/training/datasets/detection_datasets/yolo_format_detection.py
which should be changed from only allows spaces as delimiters:

label_id, cx, cw, w, h = line.split(" ")

to allowing any kind of whitespace:

label_id, cx, cw, w, h = line.split()

The current version of YoloV5 also allows any kind of white space delimiters in label files,
so making this fix improves compatibility with label files that are correctly parsed by YoloV5.

@ofrimasad
Copy link
Collaborator

opened a bug.

@ofrimasad ofrimasad added the 🐛 Bug Something isn't working label Jun 14, 2023
@ofrimasad ofrimasad self-assigned this Jun 14, 2023
@ofrimasad
Copy link
Collaborator

fixed #1173

@harpreetsahota204
Copy link

Hi @michael-mayo

Thanks for opening an issue for SG. I'm gathering some feedback on SuperGradients and YOLO-NAS.

Would you be down for a quick call to chat about your experience?

If a call doesn't work for you, no worries. I've got a short survey you could fill out: https://bit.ly/sgyn-feedback.

I know you’re super busy, but your input will help us shape the direction of SuperGradients and make it as useful as possible for you.

I appreciate your time and feedback. Let me know what works for you.

Cheers,

Harpreet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants