Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Directory handling #110

Open
arnodelorme opened this issue Sep 8, 2022 · 3 comments
Open

Directory handling #110

arnodelorme opened this issue Sep 8, 2022 · 3 comments
Assignees

Comments

@arnodelorme
Copy link
Contributor

The default directory should always be the derivative folder created by LIMO. There should be an option to go back to that folder instead of creating new folder in the current folder.

When a test already exists "one_sample_ttest", it should create a new directory and inform the user.

Every time a directory is created the user should be notified.

@CPernet
Copy link
Contributor

CPernet commented Sep 8, 2022

you may want to overwrite, so instead it asks what to do - it a design choice, I do not agree with you

@arnodelorme
Copy link
Contributor Author

Are you OK if I implement an option in that sense? You can enable or disable it.

@CPernet
Copy link
Contributor

CPernet commented Sep 9, 2022

i mean it's the default in the GUI for the last 5 years -- but of course command line will overwrite 100% of the time -- we could move the code from the GUI ~https://github.com/LIMO-EEG-Toolbox/limo_tools/blob/master/limo_random_effect.m#L279 into https://github.com/LIMO-EEG-Toolbox/limo_tools/blob/master/limo_random_select.m#L98 - but now there will be a pop up (your favourite I know) even if you don't use GUI, or we need yet an extra parameter in limo_random_select

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants