The Wayback Machine - https://web.archive.org/web/20200916131051/https://github.com/TheAlgorithms/C/pull/547
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

added taylor series #547

Open
wants to merge 1 commit into
base: master
from
Open

added taylor series #547

wants to merge 1 commit into from

Conversation

@yogesh7132
Copy link

yogesh7132 commented Jun 17, 2020

Program to implement sum of Taylor Series(e^x)

@kvedala
Copy link
Collaborator

kvedala commented Jul 4, 2020

@yogesh7132 This is a specific implementation of approximation for e^x using Taylor series expansion unto n terms. Though good, it has some fundamental errors:

  1. the factorial computation uses double precision arithmetics and hence already suffers from overflows and rounding off-errors because factorial is supposed to be an integer. To check, try to print the value of fact after 10 terms.
  2. The implementation limits should be clearly specified and thus show how the code and the algorithm implementation are valuable.

Please update the code with documentation and standards per the repo. See for example: this code from repo with documentation so that the code is self-explanatory and that the auto-tools can update the documentation website. Dont have to necessarily provide with graphs, but at least the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.