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

check_git_status() improvements #1916

Merged
merged 10 commits into from
Jan 13, 2021
Merged

check_git_status() improvements #1916

merged 10 commits into from
Jan 13, 2021

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 13, 2021

Enhanced git status checks and messages.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Clean-up in training script and improved check for repository updates.

πŸ“Š Key Changes

  • Removed unnecessary warning message related to hyperparameter renaming.
  • Simplified logging of hyperparameters in the training script.
  • Introduced a new function check_online to verify internet connectivity.
  • Refined the git status check to suggest updates for YOLOv5 repository.

🎯 Purpose & Impact

  • Enhances the usability of training scripts by omitting outdated compatibility warnings. πŸ‘
  • Streamlines user interface by normalizing the logging capitalization. πŸ”€
  • The check_online function allows for more robust detection of internet access, improving the functionality of remote operations. 🌐
  • Improved git status checking keeps users better informed about the need to pull updates, ensuring they have the latest features and bug fixes. πŸ”„

@glenn-jocher glenn-jocher changed the title Check online check_online() Jan 13, 2021
@glenn-jocher glenn-jocher changed the title check_online() check_git_status() improvements Jan 13, 2021
@glenn-jocher glenn-jocher merged commit 509dd51 into master Jan 13, 2021
@glenn-jocher glenn-jocher deleted the check_online branch January 13, 2021 05:51
KMint1819 pushed a commit to KMint1819/yolov5 that referenced this pull request May 12, 2021
* check_online()

* Update general.py

* update check_git_status()

* reverse rev-parse order

* fetch

* improved responsiveness

* comment

* comment

* remove hyp['giou'] compat warning
taicaile pushed a commit to taicaile/yolov5 that referenced this pull request Oct 12, 2021
* check_online()

* Update general.py

* update check_git_status()

* reverse rev-parse order

* fetch

* improved responsiveness

* comment

* comment

* remove hyp['giou'] compat warning
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* check_online()

* Update general.py

* update check_git_status()

* reverse rev-parse order

* fetch

* improved responsiveness

* comment

* comment

* remove hyp['giou'] compat warning
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.

1 participant