-
-
Notifications
You must be signed in to change notification settings - Fork 16k
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 smoothing line to results.png to improve readability #11536
add smoothing line to results.png to improve readability #11536
Conversation
Signed-off-by: Peter van Lunteren <contact@pvanlunteren.com>
I'm having trouble signing the CLA. I believe that I should receive instructions how to sign via email? Would it be possible to check whats going wrong with the CLA-bot? Thanks! |
@PetervanLunteren hello there! I'm sorry to hear that you're having trouble signing the CLA. The CLA bot usually sends an email with instructions on how to sign after you open the pull request. If you haven't received an email, it's possible that it was filtered out or went to your spam folder. Here are a few things you can try:
If these solutions don't work, please let us know and we will do our best to help you resolve the issue. Thank you for your interest in contributing to YOLOv5! |
Hi Glen, Thanks for your reply.
Would it be possible to send the email manually? My email address is contact@pvanlunteren.com |
Hello @PetervanLunteren, I'm sorry to hear that you still haven't received the email to sign the CLA. Unfortunately, we are not able to manually send the email as it is an automated process handled by the CLA bot. One reason for not receiving the CLA bot email could be if the email address that you have registered with GitHub is not associated with the email you're checking. Another possible reason could be delays in the delivery of emails. To ensure that you receive the CLA email, please make sure that the registered email address on your GitHub account is correct and active, and that it matches the email address that you are checking. You could also try waiting for a few more hours in case there are any delays in delivery. Thank you for your interest in contributing to YOLOv5. Please let us know if you have any more questions or if there is anything else we can help with. |
@AyushExel PR here to update plots. What do you think? |
Looks great! |
@PetervanLunteren merging PR! Do you think you could do the same update for YOLOv3 at https://github.com/ultralytics/yolov3 and YOLOv8 at https://github.com/ultralytics/ultralytics? I'd really appreciate it to keep everything consistent 🙏 |
@glenn-jocher Will do! Thanks |
PR for YOLOv5 already merged (ultralytics/yolov5#11536). Now same PR for YOLOv3 to keep consistent.
PR for YOLOv5 already merged (ultralytics/yolov5#11536). Now same PR for this repo to keep consistent.
@PetervanLunteren great to hear, thank you for your help in keeping things consistent across the repositories! Please don't hesitate to reach out if you have any questions or run into any issues. Best regards, Glenn. |
* add smoothing line to results.png to improve readability PR for YOLOv5 already merged (ultralytics/yolov5#11536). Now same PR for YOLOv3 to keep consistent. * [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>
…#11536) Signed-off-by: Peter van Lunteren <contact@pvanlunteren.com> Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
…#11536) Signed-off-by: Peter van Lunteren <contact@pvanlunteren.com> Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
EDIT: this is the third PR for this, but now I'm signing off with my real email address in stead of an pseudo one. Signing the CLA should be OK now.
This PR contains a very small addition to the original code (+ 2 lines) and does not interfere with the machine learning process whatsoever. It adds a smoothing line on top of the original plots in the results.png. With this smoothing line I hope to improve plot readability for users and provide a better insight into when their model has the best fit, and where their model is under- or overfitted.
Before:
After:
🤖 Generated by Copilot at 13397d4
Summary
📈🔬🎨
This pull request enhances the
plot_results
function inutils/plots.py
by applying a Gaussian filter to smooth the plot lines. This makes the results easier to visualize and compare.Walkthrough
gaussian_filter1d
function to smooth plot lines inplot_results
(link, link)🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Enhanced results plotting in YOLOv5 with smoothing functionality.
📊 Key Changes
gaussian_filter1d
from scipy for data smoothing.🎯 Purpose & Impact