roboflow / roboflow/notebooks

Missing dataset name in test split path in data.yaml

Open
#82 14 comments 0 reactions 1 assignee View on GitHub

@SkalskiP is already working on this.

Since Mar 22, 2023.

bug
Dominant language
Jupyter Notebook
Stars
9.7k
Forks
1.5k
PR merge metrics
No merged PRs in 30d

Description

Search before asking
  • I have searched the Roboflow Notebooks issues and found no similar bug report.
Notebook name

YOLOv5 PyTorch Object Detection

Bug

When using the Roboflow data import, the dataset name is not written to the path of the dataset test split in the data.yaml file.

Example:

test: ../test/images
train: dataset_name/train/images
val: dataset_name/valid/images

This can lead to problems, e.g. when trying to validate on the dataset test split (with --task test):

FileNotFoundError: test: /content/yolov5/test/images does not exist

It is no problem adding the dataset name to the test path in the data.yaml file, but an inexperienced user might not know how to do this.

I don't know if there is a reason behind not adding the dataset name to the test path, if not this might be a bug.

Environment
  • Google Colab
  • OS: Ubuntu 20.04.5 LTS (Focal Fossa)
  • Python: 3.9.16
  • roboflow 0.2.34
  • YOLOv5 v7.0-120-g3e55763
Minimal Reproducible Example

No response

Additional

No response

Are you willing to submit a PR?
  • Yes I'd like to help by submitting a PR!

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.