sorts/random_normal_distribution_quicksort.py has no tests

[cclauss]

Repository commit

3

Python version (python --version)

Python 3.11.5

Dependencies version (pip freeze)

Numpy

Expected behavior

Tests.

Actual behavior

No tests.

[cclauss]

@prateekiiest

[Dev-Arhaan]

Explain a bit more about this problem. Maybe I can help.

[Avineesh28]

If its regarding coming up with viable test cases and ensure running of the algortihm, I can help out!

[cclauss]

Please read CONTRIBUTING.md to understand what this repo requires for tests.

[Rohxn16]

Hey, Im'ma take this problem up.

[cclauss]

As discussed in CONTRIBUTING.md, we do NOT assign issues. We review pull requests.

[tianyizheng02]

@cclauss I think this file should be deleted entirely. It's just an implementation of quicksort with a random pivot (which we already have), and the only difference is that the array elements themselves follow a normal distribution. For all intents and purposes, it's a duplicate of sorts/random_pivot_quick_sort.py.

[Rohxn16]

@tianyizheng02 hi, I am a beginner in open source and it is my first time contributing in a big repo like this. So I deleted the file random_normal_distribution_quicksort.py which you said is a copy of the random_pivot_quicksort.py . If I have made any mistakes please feel free to inform me, Thank you.

[tianyizheng02]

Reopening this issue because I just realized that quick_sort.py already uses a random pivot, which means that both random_normal_distribution_quicksort.py and random_pivot_quick_sort.py are duplicates and both should be deleted.