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.

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

Update absMax.py #602

Merged
merged 3 commits into from
Feb 15, 2019
Merged

Update absMax.py #602

merged 3 commits into from
Feb 15, 2019

Conversation

gpapadok
Copy link
Contributor

@gpapadok gpapadok commented Nov 9, 2018

Condition was wrong in absMax() function and in main() absVal was used on a list.

Fixed two bugs
Copy link
Member

@poyea poyea left a comment

Choose a reason for hiding this comment

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

In Python, abs() is built-in and there is no need to import it. Please change line 11 using abs() and the code will compile. Thanks - John

@jmarqsilva
Copy link

So. The correct built-in function is abs() and no need to be imported. Change line 11
Please, delete this part too : "from Maths import absVal", because it's not exist.
I run!! Look result

runfile('C:/Users/Josemar/sem título1.py', wdir='C:/Users/Josemar')
-13
-11

Thansk!

@poyea poyea merged commit 3014930 into TheAlgorithms:master Feb 15, 2019
@gpapadok gpapadok deleted the patch-3 branch February 15, 2019 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants