search

rachelss / SISRS

Site Identification from Short Read Sequences.
24 stars 15 forks source link

RAL #62

Open anderspitman opened 6 years ago

anderspitman commented 6 years ago

Also added a Command Abstract Base Class and cleaned up the other commands a bit

anderspitman commented 6 years ago

@BobLiterman @rachelss the latest commits here have an updated README. Should have everything necessary to get someone up and running from scratch. I'm sure it will need to be fleshed out. Also, I trimmed quite a bit out that we might want to add back in in some form. Let me know what you think.

anderspitman commented 6 years ago

Here's a direct link to what it would look like:

https://github.com/rachelss/SISRS/tree/93f8b0b603dbbc89e2957f7a749d3b387b8ef2d8

anderspitman commented 6 years ago

Also FYI sisrs is now pip-installable:

pip install sisrs

Installs sisrs-python on the user's system.

rachelss commented 6 years ago

The title of this merge request is Do Not Merge. Is that still true?

anderspitman commented 6 years ago

Yeah I think we should wait until we're sure everyone is happy with the new README. Preferably after we've had a user test out the instructions from scratch.

anderspitman commented 6 years ago

@rachelss this should be good to merge as long as @BobLiterman signs off on it!