-
Notifications
You must be signed in to change notification settings - Fork 3
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
Remove pylab #35
base: main
Are you sure you want to change the base?
Remove pylab #35
Conversation
Had to restart the runtime session and rerun the full tutorial and then it worked ... |
There is no warning documenting that the import of numpy might fail and you have to restart the runtime!! |
Again, there is no warning documenting that the import of numpy might fail and you have to restart the runtime!!
Restarting yet again DID NOT fix. |
Restarting Runtime session and rerunning ... it worked! |
For the tutorials that worked after restarting and running again, there is no information posted that this might happen for the particular cells where it dies. In some cases there are warning but for other cells. |
Hi Kent, Thanks for your time. I think the errors on Colab are similar to what you reported (see #33). Those need to be tackled independently. I'm rather confident that there is no errors with the conda environment since we test it in CI but could you test this or with MyBinder ? |
This PR replaces
pylab
bymatlplotlib.pyplot
as pointed out in #30, #31 and #32.It also adds some warnings about the loading time of
gwpy
andmatplotlib
as pointed in the same issues.