-
Notifications
You must be signed in to change notification settings - Fork 5
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
Nathan's Solution Branch Pull Request #5
base: main
Are you sure you want to change the base?
Conversation
…uired for each package
… just outputs total required wrapping paper and ribbon
Great you've fixed your ribbon calculation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edit these Sp mistake and I'll merge the new branch in
solution_n.py
Outdated
|
||
def solution(input_data): | ||
df = pd.read_csv(input_data) | ||
list_of_presenets = df['Box Dimensions'].to_list() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
list_of_presents
solution_n.py
Outdated
combined_wrapping_paper_total = 0 | ||
combined_ribbon_total = 0 | ||
|
||
for present in list_of_presenets: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
list_of_presents
Creation of a solution_n branch and solution_n.py file to calculate the required amount of wrapping paper and ribbons needed to bring the maximum amount of joy to all the children of the world this Christmas!