corrected mb_climate_on_height for hydro month = 1 and qc as diagnostics #1351
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
when using
match_geod_pergla
, so far, the MB mean from 1999 to end of 2019 was calibrated to match the geodetic estimates. Normally it should be 2000-2019, butmb_climate_on_height
falsely used the year before y0 even if hydro_month is equal to 1. This resulted in a not perfectly calibrated mb model and was found out during the development of this notebook: https://nbviewer.org/urls/cluster.klima.uni-bremen.de/~lschuster/error_analysis/working_glacier_gdirs_comparison.ipynb#id-dmdt-dmdtda-qc3. I proposed a correction for that. All old tests of OGGM still pass with this correction. If I am right and the corrections have to be done, we need to recalibrate the preprocessed gdirs above level 2 that usematch_geod_pergla
!I also wrote a test that checks if the 20-year modelled specific mass-balance is now equal to the reference geodetic calibration data. Now it perfectly matches, however, and this is also tested inside of OGGM:
process_dummy_cru_file
to create a climate in the right time range and to test then if the calibration has worked