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

Warning messages in reports #441

Closed
adelavega opened this issue Dec 20, 2018 · 1 comment · Fixed by #725
Closed

Warning messages in reports #441

adelavega opened this issue Dec 20, 2018 · 1 comment · Fixed by #725

Comments

@adelavega
Copy link
Collaborator

Currently, report failures just show an ugly traceback.

It would be nicer and more informative if we were to catch certain types of errors (that are common) and gracefully fail, and offer suggestions.

For example: if n/as are found after transformations are applied (and thus would crash on fitlins), give a warning and suggest dropping n/as or scale + replace n/a.

@adelavega adelavega added this to To Do (Frontend) in 0.9 via automation May 14, 2019
@adelavega adelavega moved this from To Do (Frontend) to To Do (Backend) in 0.9 May 14, 2019
@adelavega
Copy link
Collaborator Author

Part of this is accomplished by wrapping the error code with a custom message with a try, catch exception.

Renaming this to focus on sending warning messages (with a new field) that can be seen by the user.

@adelavega adelavega changed the title Gracefully fail / warn in reports Warning messages in reports May 23, 2019
0.9 automation moved this from To Do (Backend) to Done Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
0.9
  
Done
Development

Successfully merging a pull request may close this issue.

1 participant