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

Add test on TravisCI for loading EEG for entire session #81

Open
mivade opened this issue Jun 26, 2018 · 1 comment
Open

Add test on TravisCI for loading EEG for entire session #81

mivade opened this issue Jun 26, 2018 · 1 comment
Labels
testing

Comments

@mivade
Copy link
Contributor

mivade commented Jun 26, 2018

There was a bug that prevented using load_eeg without any kwargs (i.e., to load an entire session). This has been fixed in an upcoming PR, but we should add a good test for this. Do we have any known very short sessions to make this test quick?
@mivade mivade added bug Something isn't working testing and removed bug Something isn't working labels Jun 26, 2018
@mivade
Copy link
Contributor Author

mivade commented Jun 28, 2018

The bug was fixed, and there is now a test when rhino is available. It would still be good to add a test with some small chunk of data, however.

@mivade mivade changed the title Add test for loading EEG for entire session Add test on TravisCI for loading EEG for entire session Jul 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mivade