-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Create histogram method for pillow #6547
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
Conversation
Hey @arisdelacruz, in your examples can you make use of actual images to showcasee the output better? Please place all the images under the media folder in docs 😄 |
I'm not sure if I hit it right this time. For review and consideration. Thanks for the guidance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @arisdelacruz, the entry looks good! Could you also please add the output images on this branch, so that we can showcase the outputs for all three examples?
Let me know once you have addressed this, thank you! 😄
I hope I hit it right. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good for a second round of review! 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The entry looks good to be merged, @arisdelacruz!
👋 @arisdelacruz 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/pillow/image/histogram Please note it may take a little while for changes to become visible. |
Description
This is an attempt to create the histogram method for pillow
Issue Solved
Closes #6525
Type of Change
Checklist
main
branch.Issues Solved
section.