Skip to content

add Error_display method to _metakernel.py #205

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

Merged
merged 2 commits into from
Aug 19, 2020
Merged

Conversation

jld23
Copy link
Contributor

@jld23 jld23 commented Aug 10, 2020

add Error_display method that will allow an error to be thrown and also display non-text objects

In the SAS Kernel, based on metakernel, there are two streams of information -- the log and the list. In parsing the log you can determine if there is an error and lag the cell as such but in a log program there could be valid output (in the form of HTML) that is helpful to the user.

This commit adds a new method to not change existing behavior.

add `Error_display` method that will all an error to be thrown and also display non-text objects

In the SAS Kernel, based on metakernel, there are two streams of informaiton -- the log and the list. In parsing the log you can determine if there is an error and lag the cell as such but in a log program there could be valid output (in the form of HTML) that is helpful to the user.

This commit adds a new method to not change existing behavior.
@coveralls
Copy link

coveralls commented Aug 10, 2020

Coverage Status

Coverage decreased (-0.4%) to 75.526% when pulling 1991ec4 on jld23:master into c0379fa on Calysto:master.

@dsblank
Copy link
Member

dsblank commented Aug 18, 2020

This looks reasonable. Could you add an example in the example directory in lieu of a test?

@dsblank
Copy link
Member

dsblank commented Aug 19, 2020

Thanks!

@dsblank dsblank merged commit 7dee16d into Calysto:master Aug 19, 2020
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

Successfully merging this pull request may close these issues.

3 participants