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

Modis data clarification and CI fixes #497

Merged
merged 17 commits into from Mar 6, 2020

Conversation

nkorinek
Copy link
Member

Changing the name of the MODIS HDF dataset to accurately represent the data inside of it, i.e., H4 data instead of H5 data.

@nkorinek
Copy link
Member Author

@lwasser @jlpalomino and I talked about this dataset and the data contained within it and we realized that the name was in fact wrong! It was a quick little fix so I figured I'd make a PR to correct the error. Let me know if I need to do anything else to this PR!

@nkorinek
Copy link
Member Author

Failed with environmental issue bc pyproj still defines crs wrong.

@nkorinek
Copy link
Member Author

docs seems to be failing due to a dead link that's accessed in make -B docs

@codecov-io
Copy link

codecov-io commented Mar 5, 2020

Codecov Report

Merging #497 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #497   +/-   ##
=====================================
  Coverage      95%    95%           
=====================================
  Files          25     25           
  Lines        1660   1660           
=====================================
  Hits         1577   1577           
  Misses         83     83
Impacted Files Coverage Δ
earthpy/io.py 53.93% <ø> (ø) ⬆️
earthpy/tests/conftest.py 100% <100%> (ø) ⬆️
earthpy/tests/test_clip.py 100% <100%> (ø) ⬆️
earthpy/tests/test_io.py 73.38% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10abaf9...1bcae4f. Read the comment docs.

@nkorinek nkorinek changed the title Modis data clarification Modis data clarification and CI fixes Mar 5, 2020
@nkorinek
Copy link
Member Author

nkorinek commented Mar 5, 2020

@lwasser CI is happy now!

Copy link

@lwasser lwasser left a comment

Choose a reason for hiding this comment

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

Awesome!! Thank you for this fix @nkorinek !! I'll bump version soonish so we can use this in class next week w the correct dirnames!

@lwasser lwasser merged commit 38d2fc5 into earthlab:master Mar 6, 2020
nkorinek added a commit to nkorinek/earthpy that referenced this pull request Mar 10, 2020
Modis data clarification and CI fixes (earthlab#497)
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.

None yet

3 participants