-
Notifications
You must be signed in to change notification settings - Fork 164
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
docs: update gradient tutorial #703
Conversation
01d7e82
to
0171ed4
Compare
PR is ready for review. Some of the code snippets didn't compile anymore, because the referenced functions either moved into the detail namespace (like Now all code snippets compile without warnings. |
Maintenance of the documentation snippets is a bit of PITA. I'm not sure if similar is feasible using reStructuredText, or we should switch over to AsciiDoc ;)
Sweet, thank you! |
Regarding this Maybe one should think to pull it out of the detail namespace again, if it is that useful for client code? I don't know the rational why it was moved into the detail namespace at all (this was done 15 years ago!). |
Good catch @marco-langer I have created a new issue #706 to address your questions in future. |
Description
This PR updates the gradient tutorial with currently used style (east-const, trailing return types, alias declarations) and fixes some other minor issues.
Currently a draft, I will update it in the next days.
Tasklist