Skip to content
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

Add docstring for undocumented gradient method #21188

Closed
wants to merge 2 commits into from

Conversation

stillyslalom
Copy link
Contributor

gradient(F,x::Vector) is useful, but not included in the manual, making method discovery difficult. Also replaced h with x in function body (since h conventionally denotes inter-point spacing rather than location).

gradient(F,x::Vector) is useful, but not included in the manual, making method discovery difficult.
@andreasnoack
Copy link
Member

See also #16113

@stillyslalom
Copy link
Contributor Author

Ah, it's part of the battery-removal project. I'm still confused as to why convenience functions are being moved from base to myriads of hard-to-discover packages without some sort of stdlib mechanism in place. I understand the maintainers' desire to trim down base, but I wish they'd slow down on migrating convenience functions out until there's a good way for new users to access basic functions without having to dig through unevenly-documented external packages.

@kshyatt kshyatt added docs This change adds or pertains to documentation linear algebra Linear algebra labels Mar 28, 2017
@fredrikekre
Copy link
Member

#23816

@StefanKarpinski StefanKarpinski added the excision Removal of code from Base or the repository label Sep 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation excision Removal of code from Base or the repository linear algebra Linear algebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants