[go: nahoru, domu]
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

[BUG] Catching general Exception may hide true cause of error #399

Open
msalvaris opened this issue Nov 6, 2019 · 0 comments
Open

[BUG] Catching general Exception may hide true cause of error #399

msalvaris opened this issue Nov 6, 2019 · 0 comments
Labels
bug Something isn't working bug-bash Issues that you came across during the bug-bash

Comments

@msalvaris
Copy link
msalvaris commented Nov 6, 2019 •
edited
Loading

Description

Here https://github.com/microsoft/ComputerVision/blob/3e0631e0dc7d5ddbfc6283b1e89b3ce51f0bd449/utils_cv/common/azureml.py#L52 you are catching a general exception which could hide the true cause of failure and make it really hard to debug. If it fails to get the workspace capture the specific exception and create the workspace.

Expected behavior (i.e. solution)

Capture specific exception
@msalvaris msalvaris added bug Something isn't working bug-bash Issues that you came across during the bug-bash labels Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working bug-bash Issues that you came across during the bug-bash
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@msalvaris