ivoflipse / Pawlabeling

Tool for processing and analyzing pressure measurements
Other
18 stars 1 forks source link

The default paths in the settings are broken #86

Closed ivoflipse closed 10 years ago

ivoflipse commented 10 years ago

When I tried to install Pawlabeling on Iris her laptop, the application crashed, because she didn't have the same path for her Dropbox folder (silly her).

So perhaps check if the folder exists, if not default to some relative path and/or query the user to change the path.

ivoflipse commented 10 years ago

Apparently I didn't check the save_settings function after I revamped my settings. Anyway, it seems to be saving again. I removed colors and contact_dict as functions, since they were only interfering with the workflow.

I also ended up making a change to io.get_file_paths, to no longer filter out anything that's not a file, because it would filter out directories and give an empty tree. So now I added icon's to highlight what's what: image

To deal with these changes I had to add a os.path.isfile check to add_measurements and change a test case that checked for empty folders.

ivoflipse commented 10 years ago

I added the checks, though I haven't verified whether they really work as intended. I also changed an absolute path to a relative one, which should prevent this from happening again.