Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

eegDb should check for the presence of pop_eegfiltnew #37

Closed
mmagnuski opened this issue Aug 27, 2014 · 2 comments
Closed

eegDb should check for the presence of pop_eegfiltnew #37

mmagnuski opened this issue Aug 27, 2014 · 2 comments
Labels
bug
Milestone
v 0.1

Comments

@mmagnuski
Copy link
Owner

pop_eegfiltnew is necessary for filtering but is not being checked for correctly.
This should be done in eeg_path or some new function for checking dependencies...
❓
@mmagnuski mmagnuski added this to the v 0.1 milestone Aug 27, 2014
@mmagnuski mmagnuski mentioned this issue Sep 13, 2014
Merged
4 tasks
mmagnuski referenced this issue Sep 13, 2014
@mmagnuski
adds test for presence of newfilt
894add1
@mmagnuski mmagnuski removed the decide label Nov 15, 2014
@mmagnuski
Copy link
Owner Author

pop_eegfiltnew is now checked for correctly but using persistent variable causes occasional problems - when one uses eeg_path('rem') to remove eeglab paths and runs eegfiltnew checker then - the persistent variable is fooling recoverEEG

@mmagnuski mmagnuski reopened this Nov 15, 2014
@mmagnuski
Copy link
Owner Author

Persistency has been removed some time ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v 0.1
Development

No branches or pull requests
1 participant
@mmagnuski