ilogue /
niprov
| Conditions | 11 |
| Total Lines | 64 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
Complex classes like FifFile.inspect() often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
| 1 | from __future__ import division |
||
| 15 | def inspect(self): |
||
| 16 | provenance = super(FifFile, self).inspect() |
||
| 17 | """ try: |
||
| 18 | img = self.libs.mne.io.Raw(self.path, allow_maxshield=True) |
||
| 19 | except ValueError: |
||
| 20 | pass |
||
| 21 | else: |
||
| 22 | inspect file |
||
| 23 | Return |
||
| 24 | """ |
||
| 25 | ftypes = { |
||
| 26 | 'cov': self.libs.mne.read_cov, |
||
| 27 | 'epo': self.libs.mne.read_epochs, |
||
| 28 | 'ave': self.libs.mne.read_evokeds, |
||
| 29 | 'fwd': self.libs.mne.read_forward_solution, |
||
| 30 | 'trans': self.libs.mne.read_trans, |
||
| 31 | 'raw': partial(self.libs.mne.io.read_raw_fif, allow_maxshield=True), |
||
| 32 | 'proj': self.libs.mne.read_proj, |
||
| 33 | } |
||
| 34 | oldLevel = logging.getLogger('mne').getEffectiveLevel() |
||
| 35 | logging.getLogger('mne').setLevel(logging.ERROR) |
||
| 36 | for ftype, readfif in ftypes.items(): |
||
| 37 | try: |
||
| 38 | img = readfif(self.path) |
||
| 39 | if img == []: |
||
| 40 | continue |
||
| 41 | break |
||
| 42 | except (ValueError, IOError): |
||
| 43 | continue |
||
| 44 | else: |
||
| 45 | ftype = 'other' |
||
| 46 | logging.getLogger('mne').setLevel(oldLevel) |
||
| 47 | |||
| 48 | if ftype == 'raw': |
||
| 49 | sub = img.info['subject_info'] |
||
| 50 | if sub is not None: |
||
| 51 | provenance['subject'] = sub['first_name']+' '+sub['last_name'] |
||
| 52 | provenance['project'] = img.info['proj_name'] |
||
| 53 | acqTS = img.info['meas_date'][0] |
||
| 54 | provenance['acquired'] = datetime.fromtimestamp(acqTS) |
||
| 55 | T = img.last_samp - img.first_samp + 1 |
||
| 56 | provenance['dimensions'] = [img.info['nchan'], T] |
||
| 57 | provenance['sampling-frequency'] = img.info['sfreq'] |
||
| 58 | provenance['duration'] = T/img.info['sfreq'] |
||
| 59 | |||
| 60 | if ftype == 'epo': |
||
| 61 | provenance['lowpass'] = img.info['lowpass'] |
||
| 62 | provenance['highpass'] = img.info['highpass'] |
||
| 63 | provenance['bad-channels'] = img.info['bads'] |
||
| 64 | provenance['dimensions'] = [img.events.shape[0], img.times.shape[0]] |
||
| 65 | |||
| 66 | if ftype == 'ave': |
||
| 67 | nEvokeds = len(img) |
||
| 68 | provenance['dimensions'] = [nEvokeds] + list(img[0].data.shape) |
||
| 69 | |||
| 70 | if ftype == 'cov': |
||
| 71 | provenance['dimensions'] = list(img.data.shape) |
||
| 72 | |||
| 73 | if ftype == 'proj': |
||
| 74 | provenance['projection-description'] = img['desc'] |
||
| 75 | |||
| 76 | provenance['fif-type'] = ftype |
||
| 77 | provenance['modality'] = 'MEG' |
||
| 78 | return provenance |
||
| 79 | |||
| 99 |
