Skip to content

Qc tutorial edits#1940

Open
joshqsumner wants to merge 52 commits into
v5.0from
qc-tutorial-edits
Open

Qc tutorial edits#1940
joshqsumner wants to merge 52 commits into
v5.0from
qc-tutorial-edits

Conversation

@joshqsumner

Copy link
Copy Markdown
Contributor

Describe your changes
Edits made to qc submodule when putting together the tutorial. Some of these are fixes for problems in previous PRs in this chain

Type of update
This is code cleanup/enhancements/bug fixes but only on the merge-in-order chain.

Associated issues
See #1910

Additional context
Merge after #1935

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 this to the PlantCV v5.0 milestone May 28, 2026
@joshqsumner joshqsumner added the merge in order Set of PRs that need to be merged sequentially label May 28, 2026
@joshqsumner joshqsumner changed the base branch from main to v5.0 May 28, 2026 14:04
@deepsource-io

deepsource-io Bot commented May 28, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 4026f6b...547ba7b 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  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Jun 30, 2026 7:16p.m. Review ↗
Code coverage Jun 30, 2026 7:46p.m. Review ↗

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 and others added 28 commits May 29, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Code quality updates 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