MRG, MAINT: Remove PySurfer from CircleCI

[larsoner]

@GuillaumeFavelier this should work at this point, right? CircleCI should maybe give us the answer.

[GuillaumeFavelier]

this should work at this point, right?

In theory yes, I'm still interested by the results

[codecov]

Codecov Report

Merging #7275 into master will increase coverage by 1.38%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #7275      +/-   ##
=========================================
+ Coverage   88.41%   89.8%   +1.38%     
=========================================
  Files         447     447              
  Lines       80597   80598       +1     
  Branches    12878   12878              
=========================================
+ Hits        71260   72379    +1119     
+ Misses       6518    5391    -1127     
- Partials     2819    2828       +9

[GuillaumeFavelier]

This made me realize that plot_parcellation.py uses PySurfer explicity:

https://17778-1301584-gh.circle-artifacts.com/0/dev/auto_examples/visualization/plot_parcellation.html#sphx-glr-auto-examples-visualization-plot-parcellation-py

But why Circle was happy?

[larsoner]

Ahh there is still pip install -r requirements.txt which has PySurfer in there.

[GuillaumeFavelier]

It seems like refactor is needed for plot_parcellation.py and plot_vector_source_estimates() (work in progress in #7060)

[larsoner]

For plot_parcellation we can probably just change it to iterate over the labels and do brain.add_label for each one. I'll close for now but I'll add to your TODO list to do this once vector source estimates are taken care of