-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
ENH: __len__ method for epochs #253
Comments
It could be len(events) as default and then get updated as bad epochs are dropped to be aligned with the shape of the data drawn from .get_data ... just throwing thoughts. |
-1, as it depends on whether bad epochs have been dropped and dropping bad epochs first is not the usual procedure in the first place |
I see, makes sense. Certainly not game changing just was thinking about it. On Wed, Dec 12, 2012 at 1:57 PM, Martin Luessi notifications@github.comwrote:
|
-1 too. We had it before and we removed it exactly because of the problem of bad epochs. Closing this. |
sure, good we discussed it ;-) On Wed, Dec 12, 2012 at 2:19 PM, Alexandre Gramfort <
|
It's really a tiny one but I would love to say: len(epochs) and get len(epochs.events).
Any opinions / objections?
The text was updated successfully, but these errors were encountered: