Add discriminability (one sample and two sample)

[jdey4]

Closes #13.

Added functionality for one sample test (@sampan501 ). This performs a one-sample test for whether the discriminability differs from random chance, as described in: https://www.biorxiv.org/content/10.1101/802629v1.full

[codecov]

Codecov Report

Merging #27 into master will decrease coverage by 0.31%.
The diff coverage is 93.1%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   94.97%   94.66%   -0.32%     
==========================================
  Files          18       22       +4     
  Lines         717      862     +145     
==========================================
+ Hits          681      816     +135     
- Misses         36       46      +10

Impacted Files	Coverage Δ
mgc/discrim/_utils.py	100% <100%> (ø)
mgc/discrim/discrimOneSample.py	84.84% <84.84%> (ø)
mgc/discrim/base.py	91.89% <91.89%> (ø)
mgc/discrim/discrimTwoSample.py	96.61% <96.61%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f57737...b18efb5. Read the comment docs.

[sampan501]

@jdey4 please add unit tests, code coverage is way to low

[jdey4]

@sampan501 added unit tests for oneSample test.

[bdpedigo]

I'd make the PR title more descriptive, even if this is a work in progress

[sampan501]

Also, far more unit tests are needed to maintain proper code coverage

[sampan501]

Try rerunning Travis to reset cache

[netlify]

Deploy preview for mgc-new ready!

Built with commit 2fd2f67

https://deploy-preview-27--mgc-new.netlify.com

[sampan501]

@jdey4 make sure you run black on all files to format them. More info can be found here: https://black.readthedocs.io/en/stable/

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@470bb73). Click here to learn what that means.
The diff coverage is 87.57%.

@@            Coverage Diff            @@
##             master      #27   +/-   ##
=========================================
  Coverage          ?   85.47%           
=========================================
  Files             ?       26           
  Lines             ?     1012           
  Branches          ?        0           
=========================================
  Hits              ?      865           
  Misses            ?      147           
  Partials          ?        0

Impacted Files	Coverage Δ
mgc/discrim/discrim_one_samp.py	100% <100%> (ø)
mgc/discrim/base.py	100% <100%> (ø)
mgc/discrim/discrim_two_samp.py	67.24% <67.24%> (ø)
mgc/discrim/_utils.py	97.82% <97.82%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 470bb73...2fd2f67. Read the comment docs.

[jdey4]

@sampan501 could you have a look?

[sampan501]

Mostly minor docs changes, 1 section with nested for loops can be compiled with numba.
@jdey4 Please add a link to the commit and line number under each comment of the review where the change has been made

[jdey4]

@sampan501 I could not add Numba and it seems that it needs broadcasting a large argument with Numba if I write a function to do the looping. It may seriously increase runtime.

[sampan501]

@sampan501 I could not add Numba and it seems that it needs broadcasting a large argument with Numba if I write a function to do the looping. It may seriously increase runtime.

That is simply not true. Numba is a jit-compiler and increases speed of nested for loops and numpy operations. See https://numba.pydata.org/numba-doc/latest/user/5minguide.html

[sampan501]

Also can you link where you addressed my comments in the commits? It is not clear at all based on the names of your commits