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

ultralytics 8.1.3 ResNet models and lighter dependencies #7700

Merged
merged 8 commits into from
Jan 19, 2024

Conversation

glenn-jocher
Copy link
Member

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

@kalenmike @hassaanfarooq01 removes the autoinstall that is happening in 8.1.2 for hub-sdk for all users.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

PR introduces updates to the GitHub workflows, documentation, model configurations, model loading, and session handling, and improves keypoint plotting functionality.

📊 Key Changes

  • GitHub workflow updated for GPT-4 PR summary formatting.
  • Documentation comments for the language drop-down menu were adjusted for clarity.
  • Package version incremented to 8.1.3.
  • New YOLOv8 image classification model configuration files added for ResNet101 and ResNet50.
  • Model initialization in engine.model.py has an additional verbose parameter for detailed logging.
  • Conditional import and requirements checking for the Ultralytics HUB SDK.
  • Proper handling of exceptions in HUB session and model loading.
  • Keypoint plotting in utils.plotting.py now also supports 2D keypoints.

🎯 Purpose & Impact

  • Workflow & documentation changes ensure consistency and clearer code maintenance.
  • Version increment signals new features and minor bug fixes.
  • New model configurations enable classification tasks using different ResNet backbones.
  • Verbose mode allows for better debugging and more transparency during model initialization and loading.
  • Exception handling improvements ensure smoother interactions with Ultralytics HUB SDK, enhancing user experience.
  • Updated keypoint plotting provides broader support for visualization in different machine learning tasks, such as pose estimation.

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

sentry-io bot commented Jan 19, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: ultralytics/engine/model.py

Function Unhandled Issue
train RuntimeError: Dataset '2024-01-19-13-18-25/dataset.yaml' error ❌ '2024-01-19-13-18-25/dataset.yaml' does not exist ...
Event Count: 232
train **RuntimeError: Dataset '2024-01-19-13-18-25/dataset.yaml' error ❌ ** ultralytics.engine.tr...
Event Count: 204
train RuntimeError: Dataset 'tv.yaml' error ❌ 'tv.yaml' does not exist ultralytics.engine.tra...
Event Count: 138
train **RuntimeError: Dataset 'trainset/data.yaml' error ❌ ** ultralytics.engine.trainer in __ini...
Event Count: 76
train RuntimeError: Dataset '/home/vamsi/Pictures/demo.yaml' error ❌ mapping values are not allowed here ...
Event Count: 38
📄 File: ultralytics/hub/session.py (Click to Expand)
Function Unhandled Issue
__init__ AttributeError: 'PosixPath' object has no attribute 'startswith' ultralytics.hub.session in _...
Event Count: 5
---

Did you find this useful? React with a 👍 or 👎

VM8198 and others added 2 commits January 19, 2024 23:39
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
Co-authored-by: Laughing-q <1185102784@qq.com>
Co-authored-by: Laughing <61612323+Laughing-q@users.noreply.github.com>
Copy link

codecov bot commented Jan 19, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c13315b) 77.19% compared to head (7a36a36) 77.16%.

Files Patch % Lines
ultralytics/engine/model.py 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7700      +/-   ##
==========================================
- Coverage   77.19%   77.16%   -0.03%     
==========================================
  Files         117      117              
  Lines       14336    14336              
==========================================
- Hits        11066    11063       -3     
- Misses       3270     3273       +3     
Flag Coverage Δ
Benchmarks 35.73% <50.00%> (ø)
GPU 39.58% <62.50%> (-0.18%) ⬇️
Tests 72.19% <87.50%> (-0.14%) ⬇️

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.

prince0310 and others added 4 commits January 19, 2024 23:43
…ication (#7589)

Co-authored-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 glenn-jocher changed the title Remove hub-sdk auto-install ultralytics 8.1.3 resnet Classify models and lighter dependencies Jan 19, 2024
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher changed the title ultralytics 8.1.3 resnet Classify models and lighter dependencies ultralytics 8.1.3 ResNet models and lighter dependencies Jan 19, 2024
@glenn-jocher glenn-jocher merged commit fcc4496 into main Jan 19, 2024
11 checks passed
@glenn-jocher glenn-jocher deleted the do-not-install-hub-sdk branch January 19, 2024 23:52
gkinman pushed a commit to Octasic/ultralytics that referenced this pull request May 30, 2024
…cs#7700)

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: Vivek Malvi <vivekmalvi@raoinformationtechnology.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
Co-authored-by: iscyy <wenvoi@163.com>
Co-authored-by: Laughing-q <1185102784@qq.com>
Co-authored-by: Laughing <61612323+Laughing-q@users.noreply.github.com>
Co-authored-by: Prince kumar <85225054+prince0310@users.noreply.github.com>
puzhiyuan pushed a commit to puzhiyuan/ultralytics that referenced this pull request Oct 18, 2024
…cs#7700)

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: Vivek Malvi <vivekmalvi@raoinformationtechnology.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
Co-authored-by: iscyy <wenvoi@163.com>
Co-authored-by: Laughing-q <1185102784@qq.com>
Co-authored-by: Laughing <61612323+Laughing-q@users.noreply.github.com>
Co-authored-by: Prince kumar <85225054+prince0310@users.noreply.github.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.

4 participants