Skip to content

Update README.md#588

Merged
andreasnoack merged 1 commit intomasterfrom
mcabbott-patch-1
Aug 4, 2022
Merged

Update README.md#588
andreasnoack merged 1 commit intomasterfrom
mcabbott-patch-1

Conversation

@mcabbott
Copy link
Member

@mcabbott mcabbott commented Jul 17, 2022

This alters the readme to:

  • Show all 4 basic functions
  • Not use random numbers, but instead something interpretable
  • Link to DiffRules, and to JuliaDiff's round-up of other AD packages
  • Not define a g = x -> ForwardDiff.gradient(f, x), as it seems like poor style to define a function like this?
  • Remove note about upgrading, since earlier versions were a long time ago. (Still covered in docs. Maybe g = x -> ... was to illustrate a change.)

@codecov-commenter

This comment was marked as off-topic.

@andreasnoack andreasnoack merged commit f365d41 into master Aug 4, 2022
@andreasnoack andreasnoack deleted the mcabbott-patch-1 branch August 4, 2022 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants