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

fix bug in dtype_dict in hyperspectral read_data #890

Merged
merged 3 commits into from
Jun 2, 2022

Conversation

JorgeGtz
Copy link
Contributor

Describe your changes
dtype_dict in read_data.py associated np.uint64 to both 14 and 15 ENVI header data type

Type of update
Is this a:

  • Bug fix

Associated issues

Additional context

@codecov
Copy link

codecov bot commented May 12, 2022

Codecov Report

Merging #890 (1728818) into master (a9fe3d6) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #890   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          141       141           
  Lines         6004      6006    +2     
=========================================
+ Hits          6004      6006    +2     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

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

Impacted Files Coverage Δ
plantcv/plantcv/hyperspectral/read_data.py 100.00% <100.00%> (ø)

@JorgeGtz JorgeGtz added bug Bug or potential bug reports ready to review labels May 12, 2022
@nfahlgren nfahlgren merged commit 2efeec8 into master Jun 2, 2022
@nfahlgren nfahlgren deleted the bug-read_data-dtype_dict-duplication branch June 2, 2022 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug or potential bug reports ready to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants