pangeo-data / climpred

:earth_americas: Verification of weather and climate forecasts :earth_africa:
https://climpred.readthedocs.io
MIT License
227 stars 49 forks source link

reconciliate perfect-model with hindcast approach #115

Closed aaronspring closed 5 years ago

aaronspring commented 5 years ago

maybe we can combine the two approaches:

aaronspring commented 5 years ago

the underlying conflict is the different ways how metrics work and are applied:

Bushuk [2018] uses both MSE and ACC and uses different approaches as we do:

Either ... or:

Therefore, I cannot see a way how to reconciliate the two approaches here. I am in favor of letting as is.

aaronspring commented 5 years ago

another thing really needed to reconciliate is the get the PM and hindcast approach on the same input dimensions before entering metric(forecast, reference). this is currently not the case.

one approach would be to create a reference with coords lead and init out of time and then concat this a member 0 with the hindcast and then supply this into compute_perfect_model(comparison=e2c) as e2c compares the ensemble mean to member 0.

aaronspring commented 5 years ago

my big problem with all this is that unless we reconciliate compute_perfect_model with compute_reference we will also need to implement how we use all the metrics twice. I dont see a way right now how to implement a metric which is not in xskillscore into compute_reference.

bradyrx commented 5 years ago

I agree with you totally. I think there might be a way to modify compute_reference to handle probabilistic and skill scores. I will give it some thought unless you are going to suggest a method in the upcoming PR.

aaronspring commented 5 years ago

I think I will kick out control, running and reference_period from compute_perfect_model. With the bootstrapping as reference forecasts to identify significance, this is not needed anymore. This will simplify the API of compute_perfect_model.

bradyrx commented 5 years ago

Addressed in https://github.com/bradyrx/climpred/pull/128