Implemented the selection sort algo using python and java #473

john2ksonn · 2018-10-28T14:04:28Z

For short term contributors: we understand that getting your commits well
defined like we require is a hard task and takes some learning. If you
look to help without wanting to contribute long term there's no need
for you to learn this. Just drop us a message and we'll take care of brushing
up your stuff for merge!

Fixes #446

By submitting this pull request I confirm I've read and complied with the
below declarations.

I have read the Contribution guidelines and I am confident that my PR reflects them.
I have followed the commit guidelines for this project.
My code follows the standard code structure.
This pull request has a descriptive title. For example, {Tag}: Add {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.
I have mentioned the issue number correctly (with hyperlink) in this pull request description.
This pull request shall only be reviewed and merged once all the checks passes. No maintainer or supporter shall be obliged to review it before this condition is met.

After you submit your pull request, DO NOT click the 'Update Branch' button.

sangamcse · 2018-10-28T14:06:22Z

selection_sort/Python/selection_sort.py

@@ -0,0 +1,28 @@
+'''
+ ____       _           _   _             ____             _   


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -1,5 +1,5 @@ ''' - ____ _ _ _ ____ _ + ____ _ _ _ ____ _ / ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ \___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __| ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_

sangamcse · 2018-10-28T14:06:23Z

selection_sort/Python/selection_sort.py

@@ -0,0 +1,28 @@
+'''
+ ____       _           _   _             ____             _   
+/ ___|  ___| | ___  ___| |_(_) ___  _ __ / ___|  ___  _ __| |_ 


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -1,6 +1,6 @@ ''' ____ _ _ _ ____ _ -/ ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ +/ ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ \___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __| ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_ |____/ \___|_|\___|\___|\__|_|\___/|_| |_|____/ \___/|_| \__|

sangamcse · 2018-10-28T14:06:24Z

selection_sort/Python/selection_sort.py

+ ____       _           _   _             ____             _   
+/ ___|  ___| | ___  ___| |_(_) ___  _ __ / ___|  ___  _ __| |_ 
+\___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __|
+ ___) |  __| |  __| (__| |_| | (_) | | |  ___) | (_) | |  | |_ 


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -2,7 +2,7 @@ ____ _ _ _ ____ _ / ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ \___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __| - ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_ + ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_ |____/ \___|_|\___|\___|\__|_|\___/|_| |_|____/ \___/|_| \__| '''

sangamcse · 2018-10-28T14:06:28Z

selection_sort/Python/selection_sort.py

+def sort(a):
+    max_idx = len(a) - 1
+    for idx in range(0, max_idx):
+        min_idx = idx        


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -14,7 +14,7 @@ def sort(a): max_idx = len(a) - 1 for idx in range(0, max_idx): - min_idx = idx + min_idx = idx for i in range(idx+1, max_idx+1): if a[i] < a[min_idx]: min_idx = i

sangamcse · 2018-10-28T14:06:29Z

selection_sort/Python/selection_sort.py

+        for i in range(idx+1, max_idx+1):
+            if a[i] < a[min_idx]:
+                min_idx = i
+        a[idx], a[min_idx] = a[min_idx], a[idx]        


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -18,7 +18,7 @@ for i in range(idx+1, max_idx+1): if a[i] < a[min_idx]: min_idx = i - a[idx], a[min_idx] = a[min_idx], a[idx] + a[idx], a[min_idx] = a[min_idx], a[idx] return a

sangamcse · 2018-10-28T14:06:30Z

selection_sort/Python/selection_sort.py

+                min_idx = i
+        a[idx], a[min_idx] = a[min_idx], a[idx]        
+    return a
+


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -20,7 +20,7 @@ min_idx = i a[idx], a[min_idx] = a[min_idx], a[idx] return a - + if __name__ == "__main__": rand_list = random.sample(range(1000), 40)

sangamcse · 2018-10-28T14:06:31Z

selection_sort/Java/SelectionSort.java

+import java.util.Random;
+
+/*
+ ____       _           _   _             ____             _   


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Java/SelectionSort.java +++ b/tmp/tmpz6y7xvz3/selection_sort/Java/SelectionSort.java @@ -1,7 +1,7 @@ import java.util.Random; /* - ____ _ _ _ ____ _ + ____ _ _ _ ____ _ / ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ \___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __| ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_

sangamcse · 2018-10-28T14:06:31Z

selection_sort/Java/SelectionSort.java

+
+/*
+ ____       _           _   _             ____             _   
+/ ___|  ___| | ___  ___| |_(_) ___  _ __ / ___|  ___  _ __| |_ 


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Java/SelectionSort.java +++ b/tmp/tmpz6y7xvz3/selection_sort/Java/SelectionSort.java @@ -2,7 +2,7 @@ /* ____ _ _ _ ____ _ -/ ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ +/ ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ \___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __| ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_ |____/ \___|_|\___|\___|\__|_|\___/|_| |_|____/ \___/|_| \__|

sangamcse · 2018-10-28T14:06:32Z

selection_sort/Java/SelectionSort.java

+ ____       _           _   _             ____             _   
+/ ___|  ___| | ___  ___| |_(_) ___  _ __ / ___|  ___  _ __| |_ 
+\___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __|
+ ___) |  __| |  __| (__| |_| | (_) | | |  ___) | (_) | |  | |_ 


Line contains following spacing inconsistencies:

Trailing whitespaces.

Origin: SpaceConsistencyBear, Section: all.pyjava.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Java/SelectionSort.java +++ b/tmp/tmpz6y7xvz3/selection_sort/Java/SelectionSort.java @@ -4,7 +4,7 @@ ____ _ _ _ ____ _ / ___| ___| | ___ ___| |_(_) ___ _ __ / ___| ___ _ __| |_ \___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __| - ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_ + ___) | __| | __| (__| |_| | (_) | | | ___) | (_) | | | |_ |____/ \___|_|\___|\___|\__|_|\___/|_| |_|____/ \___/|_| \__| */

sangamcse · 2018-10-28T14:06:33Z

selection_sort/Python/selection_sort.py

+# date: 10.28.2018
+
+import random
+


The code does not comply to PEP8.

Origin: PEP8Bear, Section: all.autopep8.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -10,6 +10,7 @@ # date: 10.28.2018 import random + def sort(a): max_idx = len(a) - 1

sangamcse · 2018-10-28T14:06:34Z

selection_sort/Python/selection_sort.py

+def sort(a):
+    max_idx = len(a) - 1
+    for idx in range(0, max_idx):
+        min_idx = idx        


The code does not comply to PEP8.

Origin: PEP8Bear, Section: all.autopep8.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -14,7 +14,7 @@ def sort(a): max_idx = len(a) - 1 for idx in range(0, max_idx): - min_idx = idx + min_idx = idx for i in range(idx+1, max_idx+1): if a[i] < a[min_idx]: min_idx = i

sangamcse · 2018-10-28T14:06:34Z

selection_sort/Python/selection_sort.py

+        for i in range(idx+1, max_idx+1):
+            if a[i] < a[min_idx]:
+                min_idx = i
+        a[idx], a[min_idx] = a[min_idx], a[idx]        


The code does not comply to PEP8.

Origin: PEP8Bear, Section: all.autopep8.

The issue can be fixed by applying the following patch:

--- a/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py +++ b/tmp/tmpz6y7xvz3/selection_sort/Python/selection_sort.py @@ -18,9 +18,9 @@ for i in range(idx+1, max_idx+1): if a[i] < a[min_idx]: min_idx = i - a[idx], a[min_idx] = a[min_idx], a[idx] + a[idx], a[min_idx] = a[min_idx], a[idx] return a - + if __name__ == "__main__": rand_list = random.sample(range(1000), 40)

sangamcse · 2018-10-28T14:06:35Z

selection_sort/Python/selection_sort.py

@@ -0,0 +1,28 @@
+'''
+ ____       _           _   _             ____             _   


W291 trailing whitespace

Origin: PycodestyleBear (W291), Section: all.autopep8.

sangamcse · 2018-10-28T14:06:36Z

selection_sort/Python/selection_sort.py

@@ -0,0 +1,28 @@
+'''
+ ____       _           _   _             ____             _   
+/ ___|  ___| | ___  ___| |_(_) ___  _ __ / ___|  ___  _ __| |_ 


W291 trailing whitespace

Origin: PycodestyleBear (W291), Section: all.autopep8.

sangamcse · 2018-10-28T14:06:36Z

selection_sort/Python/selection_sort.py

+ ____       _           _   _             ____             _   
+/ ___|  ___| | ___  ___| |_(_) ___  _ __ / ___|  ___  _ __| |_ 
+\___ \ / _ | |/ _ \/ __| __| |/ _ \| '_ \\___ \ / _ \| '__| __|
+ ___) |  __| |  __| (__| |_| | (_) | | |  ___) | (_) | |  | |_ 


W291 trailing whitespace

Origin: PycodestyleBear (W291), Section: all.autopep8.

sangamcse · 2018-10-28T14:06:37Z

selection_sort/Python/selection_sort.py

+
+import random
+
+def sort(a):


E302 expected 2 blank lines, found 1

Origin: PycodestyleBear (E302), Section: all.autopep8.

sangamcse · 2018-10-28T14:06:38Z

selection_sort/Python/selection_sort.py

+def sort(a):
+    max_idx = len(a) - 1
+    for idx in range(0, max_idx):
+        min_idx = idx        


W291 trailing whitespace

Origin: PycodestyleBear (W291), Section: all.autopep8.

sangamcse · 2018-10-28T14:06:39Z

selection_sort/Python/selection_sort.py

+        for i in range(idx+1, max_idx+1):
+            if a[i] < a[min_idx]:
+                min_idx = i
+        a[idx], a[min_idx] = a[min_idx], a[idx]        


W291 trailing whitespace

Origin: PycodestyleBear (W291), Section: all.autopep8.

sangamcse · 2018-10-28T14:06:40Z

selection_sort/Python/selection_sort.py

+                min_idx = i
+        a[idx], a[min_idx] = a[min_idx], a[idx]        
+    return a
+


W293 blank line contains whitespace

Origin: PycodestyleBear (W293), Section: all.autopep8.

Implemented the selection sort algo using python and java

cd224d4

sangamcse added process/pending_review difficulty/low hacktoberfest HACKTOBERFEST 2020 lang/Python type/algo size/XS labels Oct 28, 2018

sangamcse reviewed Oct 28, 2018

View reviewed changes

sangamcse added invalid This doesn't seem right and removed difficulty/low hacktoberfest HACKTOBERFEST 2020 lang/Python labels Oct 28, 2018

sangamcse removed process/pending_review size/XS type/algo labels Oct 28, 2018

sangamcse closed this Oct 28, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Implemented the selection sort algo using python and java #473

Implemented the selection sort algo using python and java #473

john2ksonn commented Oct 28, 2018 •

edited

Loading

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

sangamcse Oct 28, 2018

Implemented the selection sort algo using python and java #473

Implemented the selection sort algo using python and java #473

Conversation

john2ksonn commented Oct 28, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

john2ksonn commented Oct 28, 2018 •

edited

Loading