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

feat: Add find_edges method in Image #531

Merged
merged 13 commits into from
Jan 21, 2024

Conversation

Marsmaennchen221
Copy link
Contributor

@Marsmaennchen221 Marsmaennchen221 commented Jan 19, 2024

Closes #523

Summary of Changes

Added find_edges method for the new image class. Works just like the find_edges method from the old Image class with pillow.
Fixed images with one channel for both repr and both to_file methods.
Added grayscale images that only contain one channel each.
Re-enabled Image.adjust_color_balance & Image.find_edges in the Image tutorial.
Added warning for adjust_color_balance if it is used on a grayscale image.

…e the find_edges method from the old Image class with pillow

fix: Fixed images with one channel for both repr and both to_file methods
test: Added grayscale images that only contain one channel each
@Marsmaennchen221 Marsmaennchen221 requested a review from a team as a code owner January 19, 2024 01:28
@Marsmaennchen221 Marsmaennchen221 linked an issue Jan 19, 2024 that may be closed by this pull request
Copy link
Contributor

github-actions bot commented Jan 19, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 2 0 0 0.68s
✅ PYTHON mypy 2 0 1.56s
✅ PYTHON ruff 2 0 0 0.03s
✅ REPOSITORY git_diff yes no 0.01s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

Copy link

codecov bot commented Jan 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (dba23f9) 100.00% compared to head (a667bcd) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #531   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           49        49           
  Lines         2687      2706   +19     
=========================================
+ Hits          2687      2706   +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lars-reimann lars-reimann merged commit d728eb6 into main Jan 21, 2024
8 checks passed
@lars-reimann lars-reimann deleted the 523-add-find_edges-method-in-image branch January 21, 2024 09:44
lars-reimann pushed a commit that referenced this pull request Feb 3, 2024
## [0.18.0](v0.17.1...v0.18.0) (2024-02-03)

### Features

* Add adjust_color_balance method in Image ([#530](#530)) ([dba23f9](dba23f9)), closes [#525](#525)
* Add find_edges method in Image ([#531](#531)) ([d728eb6](d728eb6)), closes [#523](#523)
* class for time series ([#508](#508)) ([73cdfb1](73cdfb1)), closes [#481](#481)
@lars-reimann
Copy link
Member

🎉 This PR is included in version 0.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lars-reimann lars-reimann added the released Included in a release label Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Included in a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add find_edges method in Image
3 participants