Revise and refactor sam.visualization

RoyalHaskoningDHV / sam

Python package for time series analysis and machine learning

MIT License

26 stars 6 forks source link

Revise and refactor sam.visualization #36

Open rubenpeters91 opened 2 years ago

rubenpeters91 commented 2 years ago

The visualization module needs some refactoring, right now it seems like a random collection of (useful) functions. This might also be an opportunity to get rid of the seaborn dependency and switch to OO-style of matplotlib

sbuergers commented 2 years ago

What did you have in mind here exactly? Is it worth removing seaborn? What is the gain of matplotlib over seaborn?