Skill class

[jsmariegaard]

Let the skill() methods return an AggregatedSkill object instead of a DataFrame. The returned object can be used for better visualization of the results e.g. plots and, styled dataframe etc and have various output methods such as to_html()

[jsmariegaard]

New features look like this:

> s.style()

> s.plot_bar('bias')

>s.plot_line('bias')

> s.plot_grid('rmse', show_numbers=False)

[ecomodeller]

Looks great.

Just make sure to show plenty of examples in the User guide.

[jsmariegaard]

I am about to close this PR. Here are some final considerations/ideas/issues:

• wrapped pandas methods (SkillDataFrame class)
  • should some of them be removed again (too many)?
  • docstrings are missing
• type hints are missing
• style()
  • should columns be renamed to bg_columns to make it clear what the columns argument is used for?
  • fmt: consider to format e.g. mape, si, cc as %

On the longer run (not this PR):

• metadata in attrs e.g. unit_text which could be used in plots
• colors: it would be nice to re-use plot colors across the package
• better formating of freq-axis label texts in plots

[jsmariegaard]

@ecomodeller, do you have any opinion on the above points?