Back to the main page.
Bug 3167 - New problems in ft_plot_sens
Status | CLOSED FIXED |
Reported | 2016-07-23 00:15:00 +0200 |
Modified | 2016-11-29 09:00:29 +0100 |
Product: | FieldTrip |
Component: | plotting |
Version: | unspecified |
Hardware: | PC |
Operating System: | Windows |
Importance: | P5 normal |
Assigned to: | Jan-Mathijs Schoffelen |
URL: | |
Tags: | |
Depends on: | |
Blocks: | |
See also: |
Vladimir Litvak - 2016-07-23 00:15:43 +0200
After the latest FT update plotting Neuromag sensors in SPM stopped working. Try figure;hs = ft_plot_sens(sens, 'chantype', 'megmag') with Neuromag grad. First hs is not generated and the function crashes. After fixing that, it still doesn't work because direction is all nans. Vladimir
Jan-Mathijs Schoffelen - 2016-08-08 13:17:37 +0200
confirmed. I can solve the second problem (with the nans), but I am not sure how to address the missing output argument. In the absence of coils being drawn, hs reflects a (single) handle to the set of points (as obtained by a call to plot3). When the coils are drawn, the subfuncion plotcoil is used, which creates a patch-object for each of the coils, hence an array of handles. How does SPM rely on the output argument of this function? can it handle an array of handles, or does it require a single handle?
Vladimir Litvak - 2016-08-08 13:38:41 +0200
We change the colour. Yes return an array and I'll make sure we can do something with it. Vladimir
Jan-Mathijs Schoffelen - 2016-08-08 13:48:47 +0200
OK, implemented with pull request 203. Note that when plotting the sensors as points hs is a handle to a lineseries object, and when the sensors are plotted as coils hs is an array of axis handles
Jan-Mathijs Schoffelen - 2016-08-09 09:45:46 +0200
According to me, this can now be closed. @Vlad: agreed?
Vladimir Litvak - 2016-08-09 09:48:04 +0200
I can't check now. Will reopen if necessary.
Jan-Mathijs Schoffelen - 2016-08-09 11:06:19 +0200
Oh, and isn't there an option in ft_plot_sens to change the color? Should we look into this?
Vladimir Litvak - 2016-08-09 11:08:18 +0200
Yes, that would be helpful. We like to make the sensors green
Jan-Mathijs Schoffelen - 2016-08-11 13:13:32 +0200
(In reply to Vladimir Litvak from comment #7) I think the option 'edgecolor' is intended to determine the color.