Skip to content

Financials #5585

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

Merged
cclauss merged 18 commits into from
Oct 26, 2021
Merged

Financials #5585

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
ed89816
feat: creates math calculations for financials
tarcisiobruni Oct 24, 2021
ff0cde7
refactor: make pull request items requirements
tarcisiobruni Oct 24, 2021
e00a875
refactor: provides type hint for parameters
tarcisiobruni Oct 24, 2021
1e5cc30
refactor: applies code review suggestions
tarcisiobruni Oct 24, 2021
522dcbf
refactor: adds more examples tests
tarcisiobruni Oct 24, 2021
5234d8d
refactor: throws ValueError instead of Exception
tarcisiobruni Oct 24, 2021
980f40d
refactor: fix formatting
tarcisiobruni Oct 24, 2021
3de2457
refactor: fix formatting
tarcisiobruni Oct 24, 2021
110da41
Update interest.py
cclauss Oct 26, 2021
c3dd88b
Update and rename financials/ABOUT.md to financial/ABOUT.md
cclauss Oct 26, 2021
c1fb8d0
Rename financials/__init__.py to financial/__init__.py
cclauss Oct 26, 2021
4807cac
Rename financials/interest.py to financial/interest.py
cclauss Oct 26, 2021
bc6304e
https://www.investopedia.com
cclauss Oct 26, 2021
d953fa3
Update __init__.py
cclauss Oct 26, 2021
f6e7408
pre-commit: Disable end-of-file-fixer
cclauss Oct 26, 2021
d551e4e
Revert change to pre-commit
cclauss Oct 26, 2021
787a9b5
Update __init__.py
poyea Oct 26, 2021
34a2b72
__init__.py
poyea Oct 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: adds more examples tests
  • Loading branch information
@tarcisiobruni
tarcisiobruni committed Oct 24, 2021
commit 522dcbfa8dd9789a7bf63cd3be6dc4a479d146f5
54 changes: 48 additions & 6 deletions financials/interest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,61 @@

def simple_interest(principle:float, daily_interest_rate:float, number_of_days_between_payments:int) -> float:
"""
>>> simple_interest(18000,0.06,3)
>>> simple_interest(18000.0,0.06,3)
3240.0
Copy link
Copy Markdown
Member

@cclauss cclauss Oct 24, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add tests dealing with zero principle, negative principle, zero interest rate, negative interest rate, zero days, negative days, etc.

Copy link
Copy Markdown
Contributor Author

@tarcisiobruni tarcisiobruni Oct 24, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cclauss done

>>> simple_interest(0.0,0.06,3)
0.0
>>> simple_interest(18000.0,0.01,10)
1800.0
>>> simple_interest(18000.0,0.0,3)
0.0
>>> simple_interest(5500.0,0.01,100)
5500.0
>>> simple_interest(10000.0,-0.06,3)
Traceback (most recent call last):
Exception: daily_interest_rate must be a positive value
>>> simple_interest(-10000.0,0.06,3)
Traceback (most recent call last):
Exception: principle must be a positive value
>>> simple_interest(5500.0,0.01,-5)
Traceback (most recent call last):
Exception: number_of_days_between_payments must be a positive value
"""
result = principle * daily_interest_rate * number_of_days_between_payments
return result
if(number_of_days_between_payments < 0):
raise Exception("number_of_days_between_payments must be a positive value")
if(daily_interest_rate < 0.0):
raise Exception("daily_interest_rate must be a positive value")
if(principle < 0.0):
raise Exception("principle must be a positive value")
return principle * daily_interest_rate * number_of_days_between_payments

def compound_interest(principle:float, nominal_annual_interest_rate_percentage:float, number_of_compounding_periods:int)-> float:
"""
>>> compound_interest(10000,0.05,3)
>>> compound_interest(10000.0,0.05,3)
1576.2500000000014
>>> compound_interest(10000.0,0.05,0)
0.0
>>> compound_interest(10000.0,0.05,1)
500.00000000000045
>>> compound_interest(0.0,0.05,3)
0.0
>>> compound_interest(10000.0,0.06,-4)
Traceback (most recent call last):
Exception: number_of_compounding_periods must be a positive value
>>> compound_interest(10000.0,-3.5,3.0)
Traceback (most recent call last):
Exception: nominal_annual_interest_rate_percentage must be a positive value
>>> compound_interest(-5500.0,0.01,5)
Traceback (most recent call last):
Exception: principle must be a positive value
"""
result = principle * ((1 + nominal_annual_interest_rate_percentage) ** number_of_compounding_periods - 1)
return result
if(number_of_compounding_periods < 0):
raise Exception("number_of_compounding_periods must be a positive value")
if(nominal_annual_interest_rate_percentage < 0.0):
raise Exception("nominal_annual_interest_rate_percentage must be a positive value")
if(principle < 0.0):
raise Exception("principle must be a positive value")
return principle * ((1 + nominal_annual_interest_rate_percentage) ** number_of_compounding_periods - 1)


if __name__ == "__main__":
Expand Down