Skip to content

Update Comet hyperlinks#10500

Merged
glenn-jocher merged 9 commits intoultralytics:masterfrom
nerdyespresso:update_comet_hyperlinks
Dec 15, 2022
Merged

Update Comet hyperlinks#10500
glenn-jocher merged 9 commits intoultralytics:masterfrom
nerdyespresso:update_comet_hyperlinks

Conversation

@nerdyespresso
Copy link
Copy Markdown
Contributor

@nerdyespresso nerdyespresso commented Dec 14, 2022

Herea are the updated hyperlinks for the YOLOv5 with Comet content

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Updated integration links for Comet in YOLOv5 documentation.

📊 Key Changes

  • Modified URLs linking to Comet's platform throughout the documentation and Jupyter notebooks.
  • Revised the link in the Comet integration section within the README and tutorial notebooks.
  • Adjusted links in the Comet Logger README, directing users to the updated sections on Comet's website.

🎯 Purpose & Impact

  • Ensures users have access to the latest resources and documentation provided by Comet.
  • Promotes a more seamless integration experience by directing users to the correct Comet pages and materials.
  • Reflects Ultralytics' commitment to keeping their documentation up-to-date and user-friendly. 📚💡

Signed-off-by: nerdyespresso <106761627+nerdyespresso@users.noreply.github.com>
Signed-off-by: nerdyespresso <106761627+nerdyespresso@users.noreply.github.com>
Signed-off-by: nerdyespresso <106761627+nerdyespresso@users.noreply.github.com>
Signed-off-by: nerdyespresso <106761627+nerdyespresso@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

👋 Hello @nerdyespresso, thank you for submitting a YOLOv5 🚀 PR! To allow your work to be integrated as seamlessly as possible, we advise you to:

  • ✅ Verify your PR is up-to-date with ultralytics/yolov5 master branch. If your PR is behind you can update your code by clicking the 'Update branch' button or by running git pull and git merge master locally.
  • ✅ Verify all YOLOv5 Continuous Integration (CI) checks are passing.
  • ✅ Reduce changes to the absolute minimum required for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." — Bruce Lee

@glenn-jocher glenn-jocher changed the title Update comet hyperlinks Update Comet hyperlinks Dec 14, 2022
@glenn-jocher glenn-jocher added the TODO High priority items label Dec 14, 2022
@glenn-jocher
Copy link
Copy Markdown
Member

@nerdyespresso thanks for the PR! Is this ready to merge or do we need additional changes? I'm flying back to Madrid today but @AyushExel can help review and merge as well in the meantime.

@glenn-jocher
Copy link
Copy Markdown
Member

@pderrenger can you paste any additional changes from our email chain here if required? Thanks!

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher
Copy link
Copy Markdown
Member

@nerdyespresso PR is merged! I think everything is correct, but please double check the links in master in case I missed anything.

Thank you for your contributions to YOLOv5 🚀 and Vision AI ⭐

@glenn-jocher glenn-jocher merged commit 1ae9194 into ultralytics:master Dec 15, 2022
@glenn-jocher glenn-jocher removed the TODO High priority items label Dec 15, 2022
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.

3 participants