#78 remove notebooks code reliant valkyrie
Resolves #78
This removes all of the notebooks and code, except for:
- The
01_introductionnotebook, which now only contains overview information about (pre)icebridge/icesat/icesat2 altimetry data. It does not provide any code for searching/accessing/harmonizing data. - The
correctionsnotebook. This remains unchanged, describing to users the importance of harmonizing the CRS of data across missions, and using a synthesized dataset as an example- We will address adding a real-world example to the
correctionsnotebook in #76
- We will address adding a real-world example to the
Check out this pull request on ![]()
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
:point_left: Launch a binder notebook on this branch for commit 6ec5ab225cbb0a4fc7faff60162f4ee8ba37648f
I will automatically update this comment whenever this PR is modified
:point_left: Launch a binder notebook on this branch for commit 5d44168c027cf39089ba283a29274dc7bbfe3049
:point_left: Launch a binder notebook on this branch for commit 4f3753e01bedfca689662a701c92657e0d3f84f9
:point_left: Launch a binder notebook on this branch for commit 50e397961882f220f32c90f1c0c9b1e1cb74ab72
:point_left: Launch a binder notebook on this branch for commit f53fed5e5eefc6681c7349c28917195659353a29
:point_left: Launch a binder notebook on this branch for commit b0f2d0a0cc59227075b4fb90c879106fced4b1e1
:point_left: Launch a binder notebook on this branch for commit e0b0db5d35489e0d0e1ff67469baa583e371cb47
:point_left: Launch a binder notebook on this branch for commit 4ac2410e76423b02405d320c0c62e653aa6c1818
:point_left: Launch a binder notebook on this branch for commit 39bc77787f08c7c608522db4a4d79085a75cfc94
:point_left: Launch a binder notebook on this branch for commit 13075d9e6bd2f62e081bebf0d0e6bc6abdff6235
PR to resolve failing tests: https://github.com/nsidc/NSIDC-Data-Tutorials/pull/81
The changes look good to me!
In the 01-introduction notebook the link to OpenAltimetry should be updated to:
https://openaltimetry.earthdatacloud.nasa.gov/data/
Otherwise that notebook looks good to me! Still need to read through corrections.
Nevermind about corrections notebook! I needed to read the description of the PR more carefully above.
In the 01-introduction notebook the link to OpenAltimetry should be updated to:
https://openaltimetry.earthdatacloud.nasa.gov/data/
Otherwise that notebook looks good to me! Still need to read through corrections.
Good catch! There were some other outdated references as well, so I fixed those too. e0b0db5
@asteiker and @trey-stafford , I wonder if the materials here could be incorporated into the iceflow notebooks. They seem to be isolated here. Alternatively, we could incorporate them into the NSIDC Data Cookbook.
A path forward might be to merge this PR, assuming we are not losing any useful information, and then open an issue in the Cookbook to incorporate the material there.
An alternative would be to incorporate the information into the UW E-Science ICESat-2 Cookbook.
That said, we could do both. There is no harm in repetition or duplication
Following discussions between myself and @asteiker, we decided to migrate the 0_introduction.ipynb and corrections.ipynb to the iceflow repository (see https://github.com/nsidc/iceflow/pull/70).
The top-level README for this repository has been updated to point users to the iceflow repository and documentation. All other iceflow-related content has been removed.