Add plotting routines
Users can import plotting routines and use them on DataFrames for data visualisation purposes, e.g.:
from postprocessor.routines.<type> import <type>
<type>(df, **args)or:
import postprocessor.routines as ppr
ppr.<type>.<type>(df, **args)I essentially copied over https://git.ecdf.ed.ac.uk/swain-lab/aliby/skeletons/-/blob/arin/scripts/users/arin/alibyplot.py, but divided each plotting type into its own file to make it more organised.
Currently, it's fully functional for any users to use, but there are issues:
- General issues:
- Code repetition, e.g.
- Defining attributes in init
- Defining Axes
- Need to call ppr.. -- would be more convenient if ppr. as if the user is using matplotlib.pyplot or seaborn.
- Defaults, e.g. plot titles, are very specific to what Arin does.
- Code repetition, e.g.
- Specific issues:
- By default, the heatmap creates Axes with the left half filled with whitespace. This doesn't affect visualisation, but it's a minor annoyance.
I plan to eliminate code repetition by applying OOP/design patterns later.
Addresses issue #14 (closed).
Merge request reports
Activity
assigned to @amuoz
I created this merge request in case there is strong demand for users to use plotting functions ASAP, but it's currently a draft (work-in-progress) because I plan to refactor it to eliminate code repetition. I'm in favour of keeping it unmerged for now while I refactor the code, but you be the judge.
added 1 commit
- d055b9bd - Absolute imports for modules in routines directory
added 3 commits
added 2 commits
added 18 commits
-
41b7baf3...c9991bf4 - 17 commits from branch
master - 32bb0515 - Merge branch 'master' into plotting-routines
-
41b7baf3...c9991bf4 - 17 commits from branch
mentioned in commit 0b4f5205
added 1 commit
- 0b4f5205 - [documentation] Describe design of plotting routines
mentioned in commit 63eb44df
mentioned in commit aliby@bf72274f
mentioned in commit aliby@a46de196
