it does, but doesnt complain about this one. also guido seems to accept it.
anyawys. If I'm the only one prefering it the way it was go ahead.

On 12.01.2013, at 22:18, Eric89GXL <notifications@github.com> wrote:

> In mne/preprocessing/ica.py:
> 
> > @@ -485,12 +482,15 @@ def sources_as_raw(self, raw, picks=None, start=None, stop=None):
> >          for i in xrange(self.n_components_):
> >              ch_names.append('ICA %03d' % (i + 1))
> >              ch_info.append(dict(ch_name='ICA %03d' % (i + 1), cal=1,
> > -                logno=i + 1, coil_type=FIFF.FIFFV_COIL_NONE,
> > -                kind=FIFF.FIFFV_MISC_CH, coord_Frame=FIFF.FIFFV_COORD_UNKNOWN,
> > -                loc=np.array([0., 0., 0., 1., 0., 0., 0., 1.,
> > -                              0., 0., 0., 1.], dtype=np.float32),
> > -                unit=FIFF.FIFF_UNIT_NONE, eeg_loc=None, range=1.0,
> > -                scanno=i + 1, unit_mul=0, coil_trans=None))
> > +                                logno=i + 1, coil_type=FIFF.FIFFV_COIL_NONE,
> > +                                kind=FIFF.FIFFV_MISC_CH,
> > +                                coord_Frame=FIFF.FIFFV_COORD_UNKNOWN,
> > +                                loc=np.array([0., 0., 0., 1., 0., 0., 0., 1.,
> > +                                              0., 0., 0., 1.],
> > +                                             dtype=np.float32),
> > +                                unit=FIFF.FIFF_UNIT_NONE, eeg_loc=None,
> > +                                range=1.0, scanno=i + 1, unit_mul=0,
> > +                                coil_trans=None))
> Does your IDE do pep8 checking? On Master, if I run pep8 mne/preprocessing/ica.py, I get:
> 
> mne/preprocessing/ica.py:29:26: E121 continuation line indentation is not a multiple of four
> mne/preprocessing/ica.py:30:26: E121 continuation line indentation is not a multiple of four
> mne/preprocessing/ica.py:79:80: E501 line too long (81 > 79 characters)
> mne/preprocessing/ica.py:162:80: E501 line too long (80 > 79 characters)
> mne/preprocessing/ica.py:174:12: E127 continuation line over-indented for visual indent
> mne/preprocessing/ica.py:263:52: E128 continuation line under-indented for visual indent
> mne/preprocessing/ica.py:320:33: E126 continuation line over-indented for hanging indent
> mne/preprocessing/ica.py:321:33: E126 continuation line over-indented for hanging indent
> mne/preprocessing/ica.py:384:32: E127 continuation line over-indented for visual indent
> mne/preprocessing/ica.py:392:51: E127 continuation line over-indented for visual indent
> mne/preprocessing/ica.py:488:17: E128 continuation line under-indented for visual indent
> mne/preprocessing/ica.py:489:17: E128 continuation line under-indented for visual indent
> mne/preprocessing/ica.py:490:17: E128 continuation line under-indented for visual indent
> mne/preprocessing/ica.py:762:80: E501 line too long (81 > 79 characters)
> mne/preprocessing/ica.py:806:32: E127 continuation line over-indented for visual indent
> mne/preprocessing/ica.py:872:40: E127 continuation line over-indented for visual indent
> Note the errors for 488-490.
> 
> —
> Reply to this email directly or view it on GitHub.
>