Skip to content

removing Euler and adjacent chapters #923

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

Merged
merged 4 commits into from
Nov 7, 2021

Conversation

leios
Copy link
Member

@leios leios commented Nov 7, 2021

After revisiting some older chapters, I have decided to remove the Euler chapter. This is because of the following reasons:

  1. The chapter was not technically wrong, but it was very misleading as to how people truly implement Euer methods i practice. I think starting from the Taylor series makes sense from a physical point of view, but not mathematically
  2. The code basically had the derivative hard-coded in, which is not how people often implement this.

I could re-work the chapter to keep the code, but I would need another chapter to link it to RK, Bakcwards Euler, or other methods. I think a full re-work is needed and is on the table in a few weeks when I am working on my next project (which will involve RK).

@ntindle
Copy link
Member

ntindle commented Nov 7, 2021

I would prefer not to delete this. I would rather unlink it on the site so that we may be able to use it later

Thoughts?

@leios
Copy link
Member Author

leios commented Nov 7, 2021

Sure, that's also fine

@leios leios merged commit be2f97d into algorithm-archivists:master Nov 7, 2021
@leios leios deleted the euler_removal branch November 7, 2021 19:41
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