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
Improve log_ploss #3964
Improve log_ploss #3964
Conversation
Many thanks! This is an nbdev project, so the changes need to be made in the source notebooks, and synced from there. |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
5b56c77
to
e0a185a
Compare
Thanks for pointing that out and I'm sorry I wasn't aware of it (it's my first post on fastai). I have moved it and hope everything is ok now. If not, please let me know |
No apologies required! Thanks for putting it in the notebook. So now you need to run |
Thanks Jeremy (I should have read the CONTRIBUTION.md first)! I now executed |
The docs are updated automatically for showing function signatures. The rest of the notebook is simply rendered as-is -- so if you have any new cell outputs, they will appear in the docs too. |
That's odd - CI is being rather fussy, and complaining that your modidx file has a difference in line endings. Perhaps you modified it manually after syncing? |
I saw that too. However, I didn't touched it and when I git-reset |
I merged the current master, but the problem with |
@jph00 Hi Jeremy, just wondering if this could be merged or if you need something else? Let me know if somethings missing! |
Thank you -- sorry I missed your update. |
Discussion:
https://forums.fast.ai/t/improving-plots-especially-plot-loss/108310
Issue:
#3963