Skip to content

Prune unused imports#5711

Merged
glenn-jocher merged 2 commits intoultralytics:masterfrom
Borda:refactor/prune-imports
Nov 19, 2021
Merged

Prune unused imports#5711
glenn-jocher merged 2 commits intoultralytics:masterfrom
Borda:refactor/prune-imports

Conversation

@Borda
Copy link
Copy Markdown
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
Copy Markdown
Member

/rebase

@glenn-jocher glenn-jocher force-pushed the refactor/prune-imports branch from 1c5f2eb to ad1b9f5 Compare November 19, 2021 12:32
@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
Copy Markdown
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.

2 participants