-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
Typo on the page example #664
Comments
Hey @naceurCRAAG, why don't you create a pull request to fix this? Here are some helpful instructions, in part generated by ChatGPT:
Editing the README of a GitHub repository that you don't own requires you to create a fork of the repository, make the desired changes to the README in your forked repository, and then submit a pull request to the original repository. Here are the step-by-step instructions:
Note: It's a good practice to communicate with the repository owner or maintainers to discuss the changes you want to make before submitting a pull request. This can help avoid conflicts and ensure that your changes are in line with the project's goals. Thanks for your interest in this package, I look forward to seeing your contribution - we all have to learn once :). Chris taught me this a few years ago. |
Hello Alex :) Happy to read you :) Thank you so much for your kindness! I will try to do what you advise. Naceur |
Let me know if you get stuck, you can also find me on the slack :) |
Yes ! I will let you know even if I succeed. I try this tomorrow since here it is too late this evening. |
Hello Alex I did as you explained to me to do Have A Nice Day Naceur |
Looks good, please submit a pull request to this repo |
Thanks Alex |
Hi @naceurCRAAG here's the link that should help you, https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request |
Dear Dejan I did what you taught me Naceur |
PRs are visible here: Your is as you know already accepted: |
Dear Chris
Could you please correct a typo found on the example of the homepage here
https://github.com/SciML/NeuralPDE.jl
in the script on the 19th line
chain = Lux.Chain(Dense(dim, 16, Lux.σ), Dense(16, 16, Flux.σ), Dense(16, 1))
replace Flux with Lux
and also add at the top of the example
using OptimizationOptimisers
since we are using ADAM in the gradient descent
I made this changes and the example works
Have A Nice Day
Naceur
The text was updated successfully, but these errors were encountered: