allow abstractvector in plot_topoplot

[vladdez]

Could you write down a context and benefits of this PR? Maybe add some checking test?

[behinger]

right now topoplot(zeros(1:128) doesnt work because it is Float64

Something like a view(dataframe[1:10,1:10]) will also not work

[vladdez]

I created two tests and they both are failing Could you check them

[behinger]

maybe specify some random positions?

[vladdez]

I though any function can work without kwargs

[vladdez]

I looked in the code In current implementation any vector should have column names View() and zeroes() will not give column names

[behinger]

how would topoplot work without positions?

[behinger]

yeah you have to wrap it with a eeg_matrix_to_dataframe - I thought you had implemented that by default - sorry

[vladdez]

Done:

• AbstractDataFrame, AbstractVectorare allowed for data
• 2 additional tests are added