Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

color_correction.py fixes #1545

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

color_correction.py fixes #1545

wants to merge 4 commits into from

Conversation

natejly
Copy link
Collaborator

@natejly natejly commented Jun 2, 2024 •
edited
Loading

Related to PYL-W0102
Removed default mutable arguments and changed changed matrix splitting

@deepsource-io deepsource.io
Copy link

deepsource-io bot commented Jun 2, 2024 •
edited
Loading

Here's the code health analysis summary for commits 5bc5535..0372f55. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython✅ Success
🎯 1 occurence resolved
View Check ↗
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗

Code Coverage Report

MetricAggregatePython
Branch Coverage100%100%
Composite Coverage99.7%99.7%
Line Coverage99.7%99.7%
New Branch Coverage100%100%
New Composite Coverage100%100%
New Line Coverage100%, ✅ Above Threshold100%, ✅ Above Threshold
💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@natejly natejly changed the title Removing list as default argument in create_color_card_mask color_correction.py fixes Jun 2, 2024
@natejly natejly self-assigned this Jun 3, 2024
@nfahlgren nfahlgren added cleanup Code quality updates ready to review labels Jul 9, 2024
@nfahlgren nfahlgren added this to the PlantCV v4.4 milestone Jul 9, 2024
@HaleySchuhl HaleySchuhl removed this from the PlantCV v4.4 milestone Sep 17, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@natejly natejly

Labels
cleanup Code quality updates ready to review
Projects
PlantCV4
Status: Pull Requests
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@natejly @nfahlgren @HaleySchuhl