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.

Sign up for GitHub

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

Jump to bottom

rounding error #12

Open
AlJohri opened this issue Dec 5, 2017 · 0 comments
Open

rounding error #12

AlJohri opened this issue Dec 5, 2017 · 0 comments

Comments

@AlJohri
Copy link

AlJohri commented Dec 5, 2017 •
edited
Loading

hey @chris1610, great article and notebook on mortgage / ammortization. I noticed that you rounded the np.pmt which sometimes throws off the number of payments. for example I was doing a 3.875% interest, 30 year, 427500 principal and I got 361 periods instead of 360. removing the rounding fixed the error.

in addition, it would be cool if you added support for PMI, property tax, and home insurance to show the total out of pocket expense in another column

thanks!

update: in addition to removing the rounding, I found i also had to add: while round(end_balance, 5) > 0:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@AlJohri