-
-
Notifications
You must be signed in to change notification settings - Fork 50
Revert JAX Content in Newton's Method #340
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
Conversation
Hi @mmcky, Could you please send me the error log, as the error did not appear in my local build? Many thanks in advance. |
@HumphreyYang if you:
you can download the execution error reports as a build artifact. |
In this case the error is
from this cell
|
Many thanks @mmcky, This is strange. I just couldn't replicate the error locally and the error is not related to the environment. Would you mind resetting the cache so that I can have another go? |
@HumphreyYang is this expected behaviour for ![]() |
Many thanks @mmcky, This is expected as it shows sometimes newton's method can fail. |
Hi @mmcky, The errors now are:
Any ideas on how should I resolve them? Many thanks in advance. |
Those errors mean there is some traceback in some output. Can you replicate this locally on a clean build? |
I am getting them locally on
and it looks like
|
@HumphreyYang I will open a separate to test build the current main to rule that out. |
… into revert-newton
|
@HumphreyYang the issue with the |
Many thanks @mmcky. I saw the PDF has been built successfully. Please merge the PR when you are ready and I will merge main to this and hand it over to @jstac! |
@HumphreyYang I merged |
Hi @jstac, This PR is ready for your review. Many thanks in advance. |
As the bug in Please feel free to go ahead and merge this before we have a new publish in |
Many thanks @HumphreyYang . @mmcky , would you mind to keep handling this one? |
|
@HumphreyYang I have triggered another build now that a new version of |
Hi @mmcky, This is ready to be merged. Please kindly review these changes and let me know if further changes are needed. Many thanks. |
thanks @HumphreyYang |
Hi @mmcky, Should we wait for the build to come alive before we publish? I will have a quick look to see what changes were brought into the lecture. Many thanks. |
@HumphreyYang I will wait until we get green ticks on the fresh cache build before triggering a publish. That currently takes 1.5 hours so I will release the update later this evening. |
@HumphreyYang please check the final build on this PR and let me know if all looks as expected. I merged to get the |
Hi @mmcky, looks like the merge got some JAX into the lecture. I will restore this branch and merge it again. Apologies for the inconvenience. |
This resolves #339.