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

FileNotFoundException Not Very Explicit #62

Open
zhuchcn opened this issue Apr 6, 2024 · 1 comment
Open

FileNotFoundException Not Very Explicit #62

zhuchcn opened this issue Apr 6, 2024 · 1 comment
Assignees

Comments

@zhuchcn
Copy link
Member

zhuchcn commented Apr 6, 2024

throw new FileNotFoundException("${file} does not exist.")

I found this error can be unclear sometimes. In my case, one of the files was given as an empty string. So All it prints out is:

ERROR ~ Unable to parse config file: 'metapipeline-DNA/module/recalibrate_BAM/./default.config'

   does not exist.

I had other cases that I had to spend a lot of time figuring out where the error came from. Maybe we can also print out what variable the path is missing.

@yashpatel6
Copy link
Collaborator

Agreed, it should be better, it'd need a slight update to the error handling to make sure the error message has the necessary information without having to pass it all to all of the functions

@yashpatel6 yashpatel6 self-assigned this Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants