sccn / EEG-BIDS

21 stars 17 forks source link

BIDS export #167

Closed jacobsen-noelle closed 3 months ago

jacobsen-noelle commented 1 year ago

The bids function gives you the option to change your output directory; however, if your directory is not an empty folder, it will delete all folder contents. Despite the pop-up warning about file deletion (I thought it would just delete things in the "BIDS" folder, not parent directory"), I accidentally deleted a lot of files on my computer. Another lab member accidentally did the same. The function also overwrites permissions. This could potentially be catastrophic if users don't have data backed-up elsewhere. Instead of wiping the directory, could the toolbox be modified to create a new BIDS export folder in whatever output directory the user chooses? The new directory could have a timestamp appended in case there is more than one BIDS folder. I think this solution would be a better user interface and prevent accidental file deletion.

dungscout96 commented 3 months ago

We now have input gui for user to decide whether they want to delete existing output directory