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

Adding unsafe flag to disable warnings while running #143

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wetnun
Copy link

@wetnun wetnun commented Dec 29, 2020

Per issue #142

@Gadgetoid
Copy link
Member

Thank you.

I'm a year and change behind the curve here, but I think the Pythonic way to do this would be to convert that warning to a warning and allow warnings to be squashed with a helper function. Python has a module explicitly for warnings that - in retrospect - we should be using. Hindsight is 2020, eh! (and so is this PR... eeek)

Warnings: https://docs.python.org/3/library/warnings.html

I don't expect you to take this on, but you deserve some kind of response and am leaving this message for future me to figure this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants