Skip to content

Conversation

@zuhairabid
Copy link

Just to make the eqn. more clear in the graph

Just to make the eqn. more clear in the graph
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@ageron
Copy link
Owner

ageron commented Mar 18, 2021

Hi @zuhairabid ,
Thanks for your PR! The animate_tangent() function is used in another place with another function, so we can't hard-code y=f'(x)=2x in the function itself. However, I like the idea, so perhaps we could instead have an extra argument fp_str for this function, with default value None, and if it's not None then we use the title "y=f'(x)=" + fp_str.
What do you think?

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.

2 participants