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

Update check_yaml() comment #4713

Merged
merged 2 commits into from
Sep 8, 2021
Merged

Update check_yaml() comment #4713

merged 2 commits into from
Sep 8, 2021

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Sep 8, 2021

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improvement to YAML file handling in utilities.

πŸ“Š Key Changes

  • Updated the check_yaml function docstring to reflect enhanced functionality.
  • The check_yaml function may now download the YAML file if needed, in addition to checking its suffix.

🎯 Purpose & Impact

  • πŸ”„ The change ensures check_yaml is more versatile, automatically handling file retrieval which simplifies user workflows.
  • πŸ” Users can expect less friction when YAML files need to be downloaded as part of their process, potentially improving the overall file management experience within the YOLOv5 ecosystem.

@glenn-jocher glenn-jocher merged commit 25a7e1d into master Sep 8, 2021
@glenn-jocher glenn-jocher deleted the update/check_yaml_comment branch September 8, 2021 14:01
CesarBazanAV pushed a commit to CesarBazanAV/yolov5 that referenced this pull request Sep 29, 2021
* Update `check_yaml()` comment

* Cleanup
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* Update `check_yaml()` comment

* Cleanup
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.

None yet

1 participant