[plot_sector_ratio] Generate sector ratio plot as subplot

[BiancaMorandi]

At the moment in the template notebook we plot the sector ratio only for the main spd and direction (the top one usually) but in this way we don't capture eventual booms/ mast twists as we don't look at other wind vanes. Need to change the plot_sector_ratio function to generate subplots.

[stephenholleran]

@BiancaMorandi wrote:

we should allow to have multiple boom_dir_1 and boom_dir_2. As sensors at different heights can be mounted on booms with different orientation. Do you think that it would be easy to fix this?

@rach185 wrote:

in analyse.sector_ratio

• Accept int or list of ints as input params

• if list: check len is equal to len wspd_1 (and wspd_2)

• Update doc string

• Add example

in plot.plot_sector_ratio

• if list: check len is equal to something sensible in sec_ratio input

• add a boom counter to iterate through if list: boom_count = 0

• for pair in sec_ratio: if boom_dir is a list, take first element in list boom_dir[0], continue through loop then increment boom_counter += 1 befor next itt. If boom_dir is an int then change nothing

• Update doc string

What would calling the function look like for this scenario?

[BiancaMorandi]

@rach185 can you please add few examples in the doc string of _plot_sector_ratio_subplot function?

[stephenholleran]

Hi @rach185,

There is a bit of inconsistency if the user wishes to return the data. In the doc string, and in other functions, the data is returned as a Dataframe. However with this, if more than one pair of anemometers is sent, a Dict is returned. I think this makes it more difficult for a user to use the returned data. Why not add a column to the Dataframe with the new distribution pair?

Thanks

cc @BiancaMorandi

[stephenholleran]

@rach185 I did a little refactoring in commit 2166427 on the return data piece.
Let me know that you are ok with this?

[rach185]

@stephenholleran
Yes, that's much more concise. Thanks