Back to the main page.

Bug 3032 - add "restore all" button to GUI for ft_rejectvisual in summary mode

Status CLOSED WONTFIX
Reported 2015-12-22 22:21:00 +0100
Modified 2019-08-10 12:41:26 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P5 enhancement
Assigned to:
URL:
Tags:
Depends on:
Blocks:
See also: http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=2978

Teresa Madsen - 2015-12-22 22:21:20 +0100

This is a feature request rather than a bug report, so I hope this is the right place to put it, and that Severity = Enhancement was the right way to indicate that. I use the summary mode ft_rejectvisual GUI to choose channels to invalidate by first hiding trials that contain artifacts across all channels, then rejecting the channels that still have unusually high kurtosis, for example. After that, I restore all the trials I've rejected, since I prefer to use partial trial rejection (pulling short clips of time out of my long trials) with various calls to ft_artifact_zvalue. I end up wasting a lot of time typing in each trial number to toggle them back on, especially since ranges (e.g., 1:10) don't even work. A "restore all" button would be greatly appreciated! Thanks, Teresa


Teresa Madsen - 2016-01-04 20:06:49 +0100

...and is there, by any chance, a more intuitive way to respond to typos than this error? In an assignment A(I) = B, the number of elements in B and I must be the same. Error in rejectvisual_summary>toggle_trials (line 348) trls(n) = str2num(cell2mat(spltrls(n))); 348 trls(n) = str2num(cell2mat(spltrls(n))); I think I got this^ just because I put an extra space at the end of the string of trial numbers.


Jan-Mathijs Schoffelen - 2016-01-12 18:16:04 +0100

alternatively, one could consider a channel selection utility as in ft_databrowser.


Jan-Mathijs Schoffelen - 2016-01-12 18:18:14 +0100

would you mind investigating whether the callback function for channel selection in ft_databrowser can be re-used in ft_rejectvisual. I am afraid that it would be more than just a copy-paste.


Jan-Mathijs Schoffelen - 2017-01-17 15:31:21 +0100

It's unclear what the status of this one is, but given the fact that we're severely underpowered in terms of people contributing to fixing issues on bugzilla, Robert and JM have decided to close the low-priority bugs for now. This in order to keep the number of open bugs manageable. Feel free to reopen it, if you are willing to move this one forward towards a more proper resolution.


Robert Oostenveld - 2019-08-10 12:35:15 +0200

This closes a whole series of bugs that have been resolved (either FIXED/WONTFIX/INVALID) for quite some time. If you disagree, please file a new issue on https://github.com/fieldtrip/fieldtrip/issues.


Robert Oostenveld - 2019-08-10 12:41:26 +0200

This closes a whole series of bugs that have been resolved (either FIXED/WONTFIX/INVALID) for quite some time. If you disagree, please file a new issue on https://github.com/fieldtrip/fieldtrip/issues.