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

Prune unused imports #5711

Merged
merged 2 commits into from
Nov 19, 2021
Merged

Conversation

Borda
Copy link
Contributor

@Borda Borda commented Nov 18, 2021

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Removing unused imports in TensorFlow interface and WandB utilities.

πŸ“Š Key Changes

  • Eliminated the unused logging import from models/tf.py.
  • Removed the unused pkg_resources (aliased as pkg) import from utils/loggers/wandb/wandb_utils.py.

🎯 Purpose & Impact

  • 🧹 Cleaner Codebase: Simplifies the code by removing unnecessary imports, making it more maintainable and potentially reducing load time.
  • πŸš€ Optimizing Performance: Every small efficiency contributes to a more streamlined experience for developers and users interacting with the YOLOv5 code.
  • 🌍 Easier Comprehension for Newcomers: Cleaner codebase can help new contributors understand the codebase more easily.

@glenn-jocher
Copy link
Member

/rebase

@glenn-jocher glenn-jocher changed the title prune unused imports Prune unused imports Nov 19, 2021
@glenn-jocher glenn-jocher merged commit eb51ffd into ultralytics:master Nov 19, 2021
@glenn-jocher
Copy link
Member

@Borda PR is merged. Thank you for your contributions to YOLOv5 πŸš€ and Vision AI ⭐

@Borda Borda deleted the refactor/prune-imports branch November 19, 2021 16:24
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* prune unused imports

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@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.

None yet

2 participants