Skip to content

Color chip comparison#1779

Open
joshqsumner wants to merge 20 commits into
mainfrom
color_chip_comparison
Open

Color chip comparison#1779
joshqsumner wants to merge 20 commits into
mainfrom
color_chip_comparison

Conversation

@joshqsumner
Copy link
Copy Markdown
Contributor

Describe your changes
Adding a color chip comparison visualization to help gauge a color card's quality.

Type of update
Is this a new feature .

Associated issues
This is part of #1770

Additional context
This is based off of the branch from PR 1759.

For the reviewer
See this page for instructions on how to review the pull request.

  • PR functionality reviewed in a Jupyter Notebook
  • All tests pass
  • Test coverage remains 100%
  • Documentation tested
  • New documentation pages added to plantcv/mkdocs.yml
  • Changes to function input/output signatures added to updating.md
  • Code reviewed
  • PR approved

@joshqsumner joshqsumner added new feature New feature ideas and solutions work in progress Mark work in progress merge in order Set of PRs that need to be merged sequentially labels Sep 8, 2025
@deepsource-io
Copy link
Copy Markdown

deepsource-io Bot commented Sep 8, 2025

DeepSource Code Review

We reviewed changes in cad43f7...e3c3c28 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python May 7, 2026 2:19p.m. Review ↗
Code coverage May 7, 2026 2:19p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
100%
[✓ above threshold]
100%
Python
100%
[✓ above threshold]
100%

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@joshqsumner joshqsumner removed the merge in order Set of PRs that need to be merged sequentially label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature New feature ideas and solutions work in progress Mark work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant