plantcv icon indicating copy to clipboard operation
plantcv copied to clipboard

Adding simple rgb tutorial to the tutorial gallergy

Open annacasto opened this issue 10 months ago • 4 comments

Describe your changes Adding a simple RGB workflow to the tutorial gallery. This tutorial is a straight-forward computer vision workflow and contains references to other workflows with options for more complexity.

Type of update Is this a:

  • New feature or feature enhancement

Associated issues None

Additional context

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

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

annacasto avatar Apr 22 '24 14:04 annacasto

Here's the code health analysis summary for commits 8e34132..f6d1781. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython✅ SuccessView 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%

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

deepsource-io[bot] avatar Apr 22 '24 14:04 deepsource-io[bot]

Keely, I updated the index.ipynb notebook with the optional color correction step, so it should be all set for final review now.

annacasto avatar Apr 22 '24 21:04 annacasto

Fixed typos and added some more context

annacasto avatar Apr 25 '24 16:04 annacasto

Fixed the title of the tutorial and added Workflow Inputs

annacasto avatar Apr 26 '24 16:04 annacasto

@annacasto could you rerun the notebook and sync it to the tutorial repo? Currently it is rendered with an error:

Screenshot 2024-05-14 at 10 35 22 AM

nfahlgren avatar May 14 '24 15:05 nfahlgren