No they probably don't but at least on my box this is the default
SUBJECTS_DIR and sometimes I simply forget to set it to the current / new
directory etc. Yes, I would throw an exception with something explicit like
'please chose SUBJECTS_DIR other than %s. can't initialize the average
subject on a Freesurfer installation path.' % os.environ['SUBJECTS_DIR']
 or so ...


On Mon, Apr 1, 2013 at 2:55 AM, Christian Brodbeck <notifications@github.com
> wrote:

> In mne/transforms/coreg.py:
>
> > +                   "create_default_subject().")
> > +            raise ValueError(err)
> > +    if mne_root is None:
> > +        mne_root = get_config('MNE_ROOT', mne_root)
> > +        if mne_root is None:
> > +            err = ("MNE_ROOT environment variable not found. Please "
> > +                   "specify the mne_root parameter in you call to "
> > +                   "create_default_subject().")
> > +            raise ValueError(err)
> > +
> > +    # make sure destination does not already exist
> > +    dest = os.path.join(subjects_dir, 'fsaverage')
> > +    if os.path.exists(dest):
> > +        err = ("Can not create fsaverage because %r already exists in "
> > +               "subjects_dir %r" % ('fsaverage', subjects_dir))
> > +        raise IOError(err)
>
> Do people actually use the freesurfer subjects_dir for their experiments?
> I did not consider that possibility, as we always create a separate mri
> folder with all mri subjects for a particular experiment. So if
> subjects_dir is the freesurfer dir, should I modify the freesurfer
> fsaverage or raise an error?
> … <#13dc31793dc0d85a_>
> On Mar 31, 2013, at 20:47, "Denis A. Engemann" <notifications@github.com>
> wrote: In mne/transforms/coreg.py: > + "create_default_subject().") > +
> raise ValueError(err) > + if mne_root is None: > + mne_root =
> get_config('MNE_ROOT', mne_root) > + if mne_root is None: > + err =
> ("MNE_ROOT environment variable not found. Please " > + "specify the
> mne_root parameter in you call to " > + "create_default_subject().") > +
> raise ValueError(err) > + > + # make sure destination does not already
> exist > + dest = os.path.join(subjects_dir, 'fsaverage') > + if
> os.path.exists(dest): > + err = ("Can not create fsaverage because %r
> already exists in " > + "subjects_dir %r" % ('fsaverage', subjects_dir)) >
> + raise IOError(err) Ok I think I'm working myself forward. I think what
> would be nice to add some pointer that SUBJECTS_DIR should not be the
> default FreeSurfer SUBJECTS_DIR. … — Reply to this email directly or view
> it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/379/files#r3597617>
> .
>