Skip to content

Auto recalculate delta e when color correcting#1931

Open
joshqsumner wants to merge 62 commits into
v5.0from
auto-recalculate-deltaE-on-correction
Open

Auto recalculate delta e when color correcting#1931
joshqsumner wants to merge 62 commits into
v5.0from
auto-recalculate-deltaE-on-correction

Conversation

@joshqsumner

Copy link
Copy Markdown
Contributor

Describe your changes
Added a helper function to plantcv.transform to rerun deltaE calculation when an image is color corrected if it was run previously (presumably on the uncorrected image). To avoid circular imports this requires a small amount of refactoring for plantcv.transform.color_correction and an additional plantcv.params.deltaE attribute that I doubt people will interact with much and plantcv.params.function_args that is undocumented and could become a hidden attribute that we use to keep track of ontology or useful details.

Type of update
This is a feature enhancement.

Associated issues
Part of #1910

Additional context
Builds off of changes in PR #1922

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

@deepsource-io

deepsource-io Bot commented May 18, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 10eaa2c...263a527 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 29, 2026 7:14p.m. Review ↗
Code coverage May 29, 2026 7:14p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
100%
[✓ above threshold]
99.9%
[▼ down 0.1% from main]
Python
100%
[✓ above threshold]
99.9%
[▼ down 0.1% from main]

➟ 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 added ready to review and removed work in progress Mark work in progress labels May 18, 2026
@joshqsumner joshqsumner mentioned this pull request May 19, 2026
8 tasks
joshqsumner and others added 23 commits May 27, 2026 09:48
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancements to existing features merge in order Set of PRs that need to be merged sequentially ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants