Skip to content

update read_geotif docstring to include cropto/cutoff#62

Open
joshqsumner wants to merge 3 commits intomainfrom
read_geotiff-docstring
Open

update read_geotif docstring to include cropto/cutoff#62
joshqsumner wants to merge 3 commits intomainfrom
read_geotiff-docstring

Conversation

@joshqsumner
Copy link
Contributor

@joshqsumner joshqsumner commented Feb 23, 2026

Describe your changes
Added remaining arguments to read_geotif docstring.

Type of update
This is documentation.

Associated issues
None

Additional context
None

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-geospatial/mkdocs.yml
  • Changes to function input/output signatures added to changelog.md
  • Code reviewed
  • PR approved

@deepsource-io
Copy link

deepsource-io bot commented Feb 23, 2026

DeepSource Code Review

We reviewed changes in 9deeb5a...4bafc2e 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 Feb 25, 2026 3:44p.m. Review ↗
Test coverage Feb 25, 2026 3:46p.m. Review ↗

Code Coverage Summary

Language Line Coverage (Overall)
Aggregate
100%
Python
100%

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

@k034b363 k034b363 self-requested a review February 23, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant