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.

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

[MRG] update function to raw_to_bids #16

Merged
merged 15 commits into from
Sep 5, 2017

Conversation

teonbrooks
Copy link
Member

No description provided.

@jasmainak
Copy link
Member

@teonbrooks can you update one example so I can try it?

thanks @MartinPerez and @jasmaniak for the .tolist() suggestion
there’s no plotting. making the name reflect what the example is doing
@teonbrooks teonbrooks changed the title [WIP] update function to raw_to_bids [MRG] update function to raw_to_bids Sep 4, 2017
@teonbrooks
Copy link
Member Author

ready for review

@jasmainak
Copy link
Member

@teonbrooks have you given up on pep8? ;P

never mind, I'll make a cleanup commit and merge :)

@jasmainak jasmainak merged commit c969cd1 into mne-tools:master Sep 5, 2017
@jasmainak
Copy link
Member

Merged. Thanks @teonbrooks

@choldgraf
Copy link
Contributor

lol this was a hilarious final few moments of this pr

@teonbrooks
Copy link
Member Author

@teonbrooks have you given up on pep8? ;P

wait? what pep8 errors?! I only write beautiful code!

@teonbrooks
Copy link
Member Author

upon further review, maybe I need to properly configure my linter ;) also, we should have ci for it!

@jasmainak
Copy link
Member

maybe I need to properly configure my linter ;)

or maybe start using Sublime text instead of fancy-pants Atom :P

@agramfort
Copy link
Member

agramfort commented Sep 6, 2017 via email

@teonbrooks teonbrooks deleted the raw_to_bids branch August 24, 2018 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants