Skip to content

Commit

Permalink
update readme and min number of signature when using --signature-name…
Browse files Browse the repository at this point in the history
…s option
  • Loading branch information
cdbope committed Jun 6, 2024
1 parent 8f87e6b commit 17f2858
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
5 changes: 4 additions & 1 deletion README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ StarSignDNA
Mutation signature analysis package
-----------------------------------

* Free software: MIT license
* Free software: `MIT license <https://opensource.org/licenses/MIT>`_
* Documentation: Link to be provided

Features
Expand All @@ -21,6 +21,9 @@ To install StarSign, run this command in your terminal::
3. cd StarSigndna-master/
4. pip install -e .

Alternatively, you can install the package directly from PyPi::

pip install starsigndna
Getting started
---------------

Expand Down
7 changes: 2 additions & 5 deletions mutational_starsign/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -450,13 +450,10 @@ def read_counts(matrix_file):
M = pd.read_csv(matrix_file, delimiter='\t')
return M

def filter_signatures_old(S, signature_names):
S = S.loc[signature_names]
return S

def filter_signatures(S, signature_names):
if len(signature_names) < 5:
raise ValueError("You must select at least 5 signature names.")
if len(signature_names) < 3:
raise ValueError("You must select at least 3 signature names.")
S = S.loc[signature_names]
return S

Expand Down

0 comments on commit 17f2858

Please sign in to comment.