Topoplot series, fresh base

[behinger]

this replaces #2, now with better history.

Also docstring + tests for the binning added.

Docstring for eeg_topoplot_series (rename!) was added as well + example

• add eeg_topoplot_series testcase

[codecov-commenter]

Codecov Report

Merging #3 (9c03afb) into master (9ad8f41) will increase coverage by 0.49%.
The diff coverage is 86.27%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   84.35%   84.84%   +0.49%     
==========================================
  Files           5        6       +1     
  Lines         147      198      +51     
==========================================
+ Hits          124      168      +44     
- Misses         23       30       +7     

Impacted Files	Coverage Δ
src/TopoPlots.jl	100.00% <ø> (ø)
src/eeg.jl	90.32% <ø> (ø)
src/eeg-series.jl	86.27% <86.27%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[behinger]

I discussed with @palday last week. He is greatly in favour of removing AlgebraOfGraphics.jl as a dependency.

To do so, we'd need to set out the grid ourselfs.
I think the following two modes are important:

row-wise with multiple rows possible

1 2 3 4 5 6
A B C D E F

continuous

1 2 3
4 5 6 
7 8 9

[behinger]

I added consistent levels + colormaps to the topoplots. Before each topoplot had an independent colormap + countour level

[behinger]

hey - if this is not the right place for this tool, then I can simply include it in UnfoldMakie.jl - I just need to know :)

[SimonDanisch]

No, I think this is the right place! :) Just didn't have a lot of time to review before the basics weren't working...
Will try to review today!

[palday]

Suggested change

[behinger]

we can remove DataFrames, but I need CategoricalArrays for the cut function

[behinger]

I checked, it wont be easy to remove all of DataFrames functionality (most importantly the functionality of stack),

[behinger]

Hi!
there is really no issue to take this into UnfoldMakie.jl - but this is now open for review for ~3 month...

Should I just take it to UnfoldMakie.jl until this is reviewed? I will do this tomorrow. I can always remove it again when this is merged.

[palday]

@behinger the big blocker at the moment is moving away from the DataFrames-based API. I can do this rewrite, but don't have time at the moment. So I guess put it in UnfoldMakie and when we get this integrated here, then you can change UnfoldMakie to just call the functionality here. Thanks for the contribution and sorry for the delay!