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.

Sign up for GitHub

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

Jump to bottom

Check whether input files/output paths exist #225

Closed
6 tasks done
fedorov opened this issue Mar 6, 2017 · 1 comment · Fixed by #254
Closed
6 tasks done

Check whether input files/output paths exist #225

fedorov opened this issue Mar 6, 2017 · 1 comment · Fixed by #254
Assignees
@che85
Labels
PM SEG SR

Comments

@fedorov
Copy link
Member

fedorov commented Mar 6, 2017 •
edited by che85
Loading

Tell users if they don't exist to help troubleshoot problems. Print absolute path for the input filenames/directories passed by the user.

  • itkimage2paramap
  • paramap2itkimage
  • itkimage2segimage
  • segimage2itkimage
  • tid1500reader
  • tid1500writer
@fedorov
Copy link
Member Author

fedorov commented May 16, 2017 •
edited
Loading

Example where user was confused (most likely), and error reporting didn't help as much as it could: #250 (comment)

@fedorov fedorov mentioned this issue May 17, 2017
Closed
che85 added a commit to che85/dcmqi that referenced this issue May 19, 2017
@che85
ENH: adding checks for existence of file/files in file list (issue QI…
49bca8a 
…ICR#225)

* cerr upon missing file/directory and EXIT_FAILURE
@che85 che85 mentioned this issue May 19, 2017
Merged
che85 added a commit to che85/dcmqi that referenced this issue May 19, 2017
@che85
ENH: extracted further methods for simplifying complexity (issue QIIC…
41064dd 
…R#225)
che85 added a commit to che85/dcmqi that referenced this issue May 23, 2017
@che85
ENH: improved readability of method names (issue QIICR#225)
77494e6
che85 added a commit to che85/dcmqi that referenced this issue May 23, 2017
@che85
ENH: adding checks for existence of path/paths in file list (issue QI…
5f95138 
…ICR#225)

* also added methods for checking if required inputs were defined
* cerr upon missing file/directory and EXIT_FAILURE
fedorov added a commit that referenced this issue May 25, 2017
@fedorov
Merge pull request #254 from che85/checkForFileExistence
77edeff 
ENH: adding checks for existence of file/files in file list (issue #225)
fedorov pushed a commit that referenced this issue Jun 29, 2017
@che85 @fedorov
ENH: adding checks for existence of path/paths in file list (issue #225)
e94dfc5 
* also added methods for checking if required inputs were defined
* cerr upon missing file/directory and EXIT_FAILURE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@che85 che85

Labels
PM SEG SR
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ENH: adding checks for existence of file/files in file list (issue #225) che85/dcmqi
2 participants
@fedorov @che85