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
1 drill 0 sum array #1
base: master
Are you sure you want to change the base?
Conversation
This reverts commit f48ea84.
@profnandaa, Kindly review my PR |
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.
Fix those.
issue-001/array_sum.py
Outdated
#!/bin/python3 | ||
|
||
def array_sum(a): | ||
|
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.
Too much unnecessary lines skipped.
issue-001/array_sum.py
Outdated
arr_sum = 0 | ||
|
||
for i in arr: | ||
arr_sum += i |
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.
This should be tabbed, and the variable here is a
not arr
:
for i in a:
arr_sum += i
Most likely you didn't test your code.
@profnandaa , I saw my mistakes. I have fixed my code. |
@Chanya-M To convert the tabs to spaces, at the bottom right corner, click on the {spaces 4}. In the options you should click on Convert Indentation to spaces and you will be good to go. |
@robercicksimiyu Thanks. Let me do that. |
It is still not working after doing that. |
No description provided.