Skip to content

Update plots.py feature_visualization path issues#5519

Merged
glenn-jocher merged 2 commits intoultralytics:masterfrom
ys31jp:patch-3
Nov 5, 2021
Merged

Update plots.py feature_visualization path issues#5519
glenn-jocher merged 2 commits intoultralytics:masterfrom
ys31jp:patch-3

Conversation

@ys31jp
Copy link
Copy Markdown
Contributor

@ys31jp ys31jp commented Nov 5, 2021

Error running python detect.py --visualize
#5503

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

This PR appears to be a placeholder or an error, as there are no code changes.

📊 Key Changes

  • None, there is no diff provided in this PR.

🎯 Purpose & Impact

  • Since there are no modifications, there is no direct impact or benefit to the users from this PR.

Error running python detect.py --visualize
ultralytics#5503
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👋 Hello @ys31jp, thank you for submitting a 🚀 PR! To allow your work to be integrated as seamlessly as possible, we advise you to:

  • ✅ Verify your PR is up-to-date with upstream/master. If your PR is behind upstream/master an automatic GitHub actions rebase may be attempted by including the /rebase command in a comment body, or by running the following code, replacing 'feature' with the name of your local branch:
git remote add upstream https://github.com/ultralytics/yolov5.git
git fetch upstream
git checkout feature  # <----- replace 'feature' with local branch name
git merge upstream/master
git push -u origin -f
  • ✅ Verify all Continuous Integration (CI) checks are passing.
  • ✅ Reduce changes to the absolute minimum required for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." -Bruce Lee

@glenn-jocher
Copy link
Copy Markdown
Member

@ys31jp thanks for the PR! This should already be fixed by the PR I recently submitted though in #5518. Can you git pull to update your code and verify that #5518 has solved the issue for you?

@glenn-jocher glenn-jocher merged commit 36a4de1 into ultralytics:master Nov 5, 2021
@ys31jp
Copy link
Copy Markdown
Contributor Author

ys31jp commented Nov 5, 2021

@glenn-jocher Hi, yes #5518 has solved my issue. Thanks a lot. Sorry for double the work.

BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* Update plots.py

Error running python detect.py --visualize
ultralytics#5503

* Update plots.py

Co-authored-by: Glenn Jocher <glenn.jocher@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.

2 participants