The GUI looks great!
My proposal for the doc / manual and the human-python interfacing would be
to more clearly describe what 'prepare fsaverage' includes and add some
verbosity like e.g. our Raw object on initialization so the user 'knows'
the schedule and it's status.
For the second command, I would add an error message like 'fsaverage not
prepared. please run set_default_subject' or so. Maybe we could even rename
it create_average_subject or so.


On Mon, Apr 1, 2013 at 2:48 AM, Denis-Alexander Engemann <
denis.engemann@gmail.com> wrote:

> It seems after setting SUBJECTS_DIR the command succeeded. Some print
> message telling what it will do and that it did the job might be reassuring
> ;-)
>
>
> On Mon, Apr 1, 2013 at 2:47 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)
>>
>>  If I remove it it will complain that it does not exist ...
>>
>> only if you remove it from freesurfer's subjects directory
>>
>> I think it would be great to have one example that works like our
>> examples.
>>
>> just for create_default_subject? It's just one command :) The only reason
>> the error happens is probably because you are using the mne sample dataset
>> which already has fsaverage or another subjects_dir which already has
>> fsaverage. You can always go with the one you already have, you might just
>> not have t
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/379/files#r3597599>
>> .
>>
>
>