Open hannah-rae opened 5 months ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
Updates:
compare_covermaps
:
get_test_years
gets rid of the TEST_CODE, per Ivan’s reviewThese changes were made to ensure the year
parameter is maintained in the extract_test
and to prevent any errors in its use case in TestCovermaps
.
Demo-notebook
(to be deleted before merge):
extract_year
only selects the nearest year from maps with multiple years_covered and updates the ee_asset string. For other single-year maps, we could select maps with the exact test year or those within a range of ±1 year of the test year. (The latter is implemented in the notebook). The analysis will focus on maps produced closest to the test set year.
This might be a use case specific to area estimates; however, let me know what you think about this for intercomparison @hannah-rae. I will include it in the area estimates notebook in a separate PR.
For maps that have more than one year available, we use the closest year to the test set year.
TODO: update results for all map notebooks