Merge AggregatedSkill and SpatialSkill classes? #286
jsmariegaard
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Would it be possible to have a single Skill class having a xr.Dataset to cover both the retun of skill(), mean_skill() and spatial_skill() ? We could then get rid of the MultiIndex 🤔. The ds should contain the meta data from the Comparer class which could be used for filtering etc (e.g. DA-station vs validation station). The coords could be observation, model, x, y, z, etc (depending on the
by
argument). The dimensions would also depend on theby
argument and in case of the gridded skill also be x and y. I guess the AggregatedSkill is just a special case of GriddedSkill with only a single "point" per model/obs...Thoughts? @ecomodeller ?
Beta Was this translation helpful? Give feedback.
All reactions