search

xiangruili / dicm2nii

DICOM to NIfTI conversion, NIfTI tools
BSD 2-Clause "Simplified" License
89 stars 45 forks source link

Change default bids gui width to 800 #48

Closed ddwagner closed 3 years ago

ddwagner commented 3 years ago

We have a study that requires a lot of manual intervention during dicm2nii conversion to bids. The gui window that pops up is quite small (default width was 420) requiring us to manually extend the window for every subject that requires editing. Feel free to reject this pull if there's a reason for setting the default width so small, but if not, I've been finding that 800 (or higher) works well as a default and we can now see all the columns.

xiangruili commented 3 years ago

@ddwagner I don't see problem with increasing width. But the idea is that you set up unique names for one subject, and rest of the subjects will convert by itself without manual intervention. Does something go wrong that requires intervention for each subject? @tanguyduval do you have any concern for this?

ddwagner commented 3 years ago

I think it will always be the case that a few subjects will require manual intervention in the case of unique aborted runs or repeat runs or things being out of order. Ideally students wouldn't export those runs, but alas we're stuck with them in our archives. These will, I assume, always trigger a gui window to allow one to set which runs to skip. In one of our studies this kind of thing shows up a lot, hence widening the window to make sure the modality column is always visible! I can't imagine anyone is running in 800x640 in this day and age but perhaps there's a case to be made for smaller windows?