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

Add psutil and ipython to requirements.txt #7998

Merged
merged 3 commits into from
May 26, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 26, 2022

Lightweight packages used by YOLOv5 for system utilization (psutil) and interactive notebooks (IPython)

๐Ÿ› ๏ธ PR Summary

Made with โค๏ธ by Ultralytics Actions

๐ŸŒŸ Summary

Enhancement of development tools within the YOLOv5 requirements.

๐Ÿ“Š Key Changes

  • ๐Ÿ“˜ Added ipython: Integration of an interactive notebook for development.
  • ๐Ÿ“Š Included psutil: Tool for monitoring system utilization.
  • ๐Ÿงฎ Repositioned thop: Moved the FLOPs (floating point operations per second) computation library within the file for better organization.

๐ŸŽฏ Purpose & Impact

  • ๐Ÿ“ˆ Interactive Development: ipython provides developers with a powerful interactive shell that can speed up the testing and debugging process.
  • ๐Ÿ“Š Resource Tracking: psutil allows users to track and monitor the performance of their system, which can help in optimizing the usage of resources when running YOLOv5.
  • ๐Ÿ” Performance Metrics Clarity: Relocating thop ensures it's grouped correctly with other extras, possibly hinting at a reorganization for better readability and maintainability of requirements.

These improvements will likely assist developers in both development efficiency and performance tracking, without impacting the end user's experience of the core functionality of YOLOv5.

Lightweight packages used by YOLOv5 for system utilization (psutil) and interactive notebooks (IPython)
@glenn-jocher glenn-jocher self-assigned this May 26, 2022
@glenn-jocher glenn-jocher merged commit 980aace into master May 26, 2022
@glenn-jocher glenn-jocher deleted the update/requirements branch May 26, 2022 17:12
tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
* Add `psutil` and `ipython` to requirements.txt

Lightweight packages used by YOLOv5 for system utilization (psutil) and interactive notebooks (IPython)

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

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

* sort alphabetically

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
* Add `psutil` and `ipython` to requirements.txt

Lightweight packages used by YOLOv5 for system utilization (psutil) and interactive notebooks (IPython)

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

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

* sort alphabetically

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

1 participant