Created problem_34 in project_euler#2305
Created problem_34 in project_euler#2305cclauss merged 23 commits intoTheAlgorithms:masterfrom Kush1101:master
Conversation
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 10bc62e0-dd5d-11ea-9bc7-ef10f4dac8ce |
Travis tests have failedHey @Kush1101, TravisBuddy Request Identifier: e262c6d0-dd5e-11ea-9bc7-ef10f4dac8ce |
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: f1cd8b80-dd60-11ea-9bc7-ef10f4dac8ce |
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 9a6b2220-dd61-11ea-9bc7-ef10f4dac8ce |
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 0b87c750-dd63-11ea-9bc7-ef10f4dac8ce |
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 6ffe9ec0-dd63-11ea-9bc7-ef10f4dac8ce |
|
How do I create the init.py file so that it passes the tests? |
|
Thank You sir for your help. Hope this gets merged 🤞. |
Incorporates the proposed changes
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 1a5d5ea0-dd6a-11ea-9bc7-ef10f4dac8ce |
project_euler/problem_34/sol1.py
Outdated
| if sum_of_digit_factorial(num) == num: | ||
| summ += num | ||
| return summ | ||
| the_list.append(num) |
There was a problem hiding this comment.
Could we make this list using a https://docs.python.org/3/tutorial/datastructures.html#list-comprehensions
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Travis tests have failedHey @Kush1101, TravisBuddy Request Identifier: dc9a1280-dd6d-11ea-9bc7-ef10f4dac8ce |
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 240dd9d0-dd6e-11ea-9bc7-ef10f4dac8ce |
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: 89579c30-dd6f-11ea-9bc7-ef10f4dac8ce |
|
Looks good to me... Should we land it?? |
Travis tests have failedHey @Kush1101, TravisBuddy Request Identifier: 6ada79c0-dd70-11ea-9bc7-ef10f4dac8ce |
|
If the tests pass then can you please merge it. |
|
@cclauss |
cclauss
left a comment
There was a problem hiding this comment.
Awesome work here! Thanks for your persistence.
|
Hey @Kush1101, TravisCI finished with status TravisBuddy Request Identifier: baf625e0-dd74-11ea-9bc7-ef10f4dac8ce |
|
@cclauss Thank you sir for your patience. |
* Create __init__.py * Add files via upload * Rename solution.py.py to solution.py * Delete __init__.py * Update and rename solution.py to sol1.py * Update sol1.py * Create __init__.py * Update __init__.py * Delete __init__.py * Add files via upload * Update __init__.py * Add #\n * Update sol1.py Incorporates the proposed changes * Update sol1.py * Update sol1.py * Update project_euler/problem_34/sol1.py Co-authored-by: Christian Clauss <cclauss@me.com> * Update project_euler/problem_34/sol1.py Co-authored-by: Christian Clauss <cclauss@me.com> * Update project_euler/problem_34/sol1.py Co-authored-by: Christian Clauss <cclauss@me.com> * Update project_euler/problem_34/sol1.py Co-authored-by: Christian Clauss <cclauss@me.com> * Update sol1.py * Update sol1.py * Update sol1.py * Use int(n) instead of floor(n) Co-authored-by: Christian Clauss <cclauss@me.com>
Describe your change:
Checklist:
Fixes: #{$ISSUE_NO}.