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 numerical anlysis #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

danielFHcode
Copy link

Added a numerical analysis (חישוב נומרי) section and adjusted some things to fit it inside 4 pages*.

*Are you sure the quarter-page footer is necessary? I mean, I didn't remove it but still.

@Sh-Pe
Copy link
Owner

Sh-Pe commented Sep 5, 2024

It's good, but:

  • I tend to explain how stuff works, rather than adding code (it's good though, maybe I'll change some stuff later on).
  • Numerical analysis isn't a part of the course for this year. For that reason, I'll merge that only after the tests will be done, for future use.

@Sh-Pe
Copy link
Owner

Sh-Pe commented Oct 30, 2024

Now the course is done, this year they'll probably need this. I don't have time to edit it so just solve the conflicts and I'll merge it.
@danielFHcode

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