Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reading TIF file #3

Merged
merged 75 commits into from
May 18, 2024
Merged

Reading TIF file #3

merged 75 commits into from
May 18, 2024

Conversation

dhiraj-ms
Copy link
Contributor

Function to read TIF file, and extracting crucial geographical and dimensional information, and converting them into a manipulable format for further analysis and processing.

Describe your changes
A clear and concise description of what changes are made by this pull request.
What was the previous functionality (if relevant) and what can we do now with
these changes.

Type of update
Is this a:

  • Bug fix
  • New feature or feature enhancement
  • Update to documentation
  • Work in progress

Associated issues
Reference associated issue numbers. Does this pull request close any issues?

Additional context
Add any other context about the problem here.

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

Function to read TIF file, and extracting crucial geographical and dimensional information, and converting them into a manipulable format for further analysis and processing.
@dhiraj-ms dhiraj-ms added the good first issue Good for newcomers label Feb 27, 2024
Copy link

codecov bot commented Feb 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (ec7f043) to head (ca2c314).
Report is 28 commits behind head on main.

Current head ca2c314 differs from pull request most recent head 17eb3ed

Please upload reports for the commit 17eb3ed to get more accurate results.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            1         1           
=========================================
  Hits             1         1           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Member

@nfahlgren nfahlgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also check the deepsource issue(s) that it flagged. Thanks!

pyproject.toml Outdated Show resolved Hide resolved
readtifimage.py Outdated Show resolved Hide resolved
Copy link

deepsource-io bot commented Mar 4, 2024

Here's the code health analysis summary for commits 1aa0983..17eb3ed. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage⚠️ Artifact not reportedTimed out: Artifact was never reportedView Check ↗

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

@HaleySchuhl HaleySchuhl removed their request for review March 11, 2024 19:48
@nfahlgren nfahlgren removed the help wanted Extra attention is needed label May 18, 2024
@nfahlgren nfahlgren merged commit 61c76c0 into danforthcenter:main May 18, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants