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

Explorer with LanceDB, Actions and Docs updates #7487

Merged
merged 8 commits into from Jan 10, 2024
Merged

Explorer with LanceDB, Actions and Docs updates #7487

merged 8 commits into from Jan 10, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 10, 2024

Thank you πŸ™ for your contribution to Ultralytics πŸš€! Your effort in enhancing our repositories is greatly appreciated. To streamline the process and assist us in integrating your Pull Request (PR) effectively, please follow these steps:

  1. Check for Existing Contributions: Before submitting, kindly explore existing PRs to ensure your contribution is unique and complementary.

  2. Link Related Issues: If your PR addresses an open issue, please link it in your submission. This helps us better understand the context and impact of your contribution.

  3. Elaborate Your Changes: Clearly articulate the purpose of your PR. Whether it's a bug fix or a new feature, a detailed description aids in a smoother integration process.

  4. Ultralytics Contributor License Agreement (CLA): To uphold the quality and integrity of our project, we require all contributors to sign the CLA. Please confirm your agreement by commenting below:

    I have read the CLA Document and I hereby sign the CLA

For more detailed guidance and best practices on contributing, refer to our βœ… Contributing Guide. Your adherence to these guidelines ensures a faster and more effective review process.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Updated pre-commit configurations, enhanced dataset explorer API documentation, and improved code readability.

πŸ“Š Key Changes

  • πŸ”„ Replaced isort and yapf in .pre-commit-config.yaml with ruff, simplifying pre-commit hooks for code formatting.
  • ✨ Enhanced documentation for the dataset explorer API (api.md) with tips and advanced usage examples.
  • πŸ“Έ Added screenshots to the dataset explorer GUI documentation (dashboard.md) to illustrate usage.
  • πŸš€ Improved explanations and added a tip regarding the functionality of Ultralytics Explorer (index.md) and its reliance on LanceDB.
  • 🎨 Created a new HTML announcement block in the documentation (main.html).
  • πŸ”§ Tweaked the pyproject.toml file to refine codespell configuration.
  • πŸ” Clarified variable names in the loss computation code (loss.py) to enhance readability.
  • 🐞 Fixed a potential bug by removing index variables from the loop (predict.py).

🎯 Purpose & Impact

  • πŸ’‘ The consolidation of code formatting tools streamlines the development process, making it easier to maintain code quality.
  • πŸ“š Enhanced documentation with powerful tips and visual aids provides a better understanding of the software, enabling users to more effectively utilize the dataset explorer.
  • πŸ”’ The announcement block serves to engage users with the latest updates and resources related to Ultralytics.
  • πŸ› οΈ Improved code readability and minor bug fixes contribute to a more robust and maintainable codebase, indirectly benefiting users by ensuring higher software reliability and quality.

glenn-jocher and others added 4 commits January 10, 2024 19:29
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests βœ…

Comparison is base (0e7221f) 77.04% compared to head (ba6a9de) 77.04%.

❗ Current head ba6a9de differs from pull request most recent head 59f14f6. Consider uploading reports for the commit 59f14f6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7487   +/-   ##
=======================================
  Coverage   77.04%   77.04%           
=======================================
  Files         117      117           
  Lines       14331    14331           
=======================================
  Hits        11042    11042           
  Misses       3289     3289           
Flag Coverage Ξ”
Benchmarks 35.55% <0.00%> (ΓΈ)
GPU 39.59% <0.00%> (ΓΈ)
Tests 72.36% <100.00%> (ΓΈ)

Flags with carried forward coverage won't be shown. Click here to find out more.

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

Kayzwer and others added 4 commits January 10, 2024 20:12
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher changed the title Update pre-commit actions Explorer, Actions and Docs updates Jan 10, 2024
@glenn-jocher glenn-jocher changed the title Explorer, Actions and Docs updates Explorer with LanceDB, Actions and Docs updates Jan 10, 2024
@glenn-jocher glenn-jocher merged commit 09ee982 into main Jan 10, 2024
8 checks passed
@glenn-jocher glenn-jocher deleted the actions branch January 10, 2024 19:30
scosijnilvo pushed a commit to scosijnilvo/yolov8 that referenced this pull request Jan 19, 2024
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: Muhammad Rizwan Munawar <chr043416@gmail.com>
Co-authored-by: Kayzwer <68285002+Kayzwer@users.noreply.github.com>
Co-authored-by: Ayush Chaurasia <ayush.chaurarsia@gmail.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
hmurari pushed a commit to hmurari/ultralytics that referenced this pull request Apr 17, 2024
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: Muhammad Rizwan Munawar <chr043416@gmail.com>
Co-authored-by: Kayzwer <68285002+Kayzwer@users.noreply.github.com>
Co-authored-by: Ayush Chaurasia <ayush.chaurarsia@gmail.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
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

4 participants