Fix cli bug - Githubissues

ME-ICA / aroma

ICA-AROMA, as a Python package. A work in progress.

Apache License 2.0

7 stars 11 forks source link

Fix cli bug #26

Closed eurunuela closed 4 years ago

eurunuela commented 4 years ago

Closes #21 .

Changes proposed in this pull request:

Make -mel_dir default to None.
Rename legacy ica-aroma-org into aroma.

eurunuela commented 4 years ago

The changes look good to me. I don't love the asymmetry between mel_dir and in_feat (i.e., mel_dir has a default set, but in_feat doesn't), but it doesn't impact behavior.

It's weird, yes. I've just made it a None just in case.