Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove some scripts/ #134

Closed
ctb opened this issue Aug 28, 2013 · 8 comments
Closed

Remove some scripts/ #134

ctb opened this issue Aug 28, 2013 · 8 comments

Comments

@ctb
Copy link
Member

ctb commented Aug 28, 2013

ec.py, edit-kmers.py, diginorm-correct.py,normalize-by-kadian.py,

Also, document and test count-median.py.

@ctb
Copy link
Member Author

ctb commented Sep 2, 2013

Actually, let's leave ec.py -- it should just be moved to sandbox/ or tested/documented, I think. @fishjord, are you using ec.py?

ctb added a commit that referenced this issue Sep 2, 2013
@fishjord
Copy link
Contributor

fishjord commented Sep 2, 2013

As a reference of what the EC strategy is kinda suppose to work like.
Ideally instead of convoluted logic about whether gaps are in the
beginning or end of the alignment I'm planning on using the truncated
alignment flag that the new read aligner returns as part of the Alignment
object

On Sun, Sep 1, 2013 at 8:01 PM, C. Titus Brown notifications@github.comwrote:

Actually, let's leave ec.py -- it should just be moved to sandbox/ or
tested/documented, I think. @fishjord https://github.com/fishjord, are
you using ec.py?


Reply to this email directly or view it on GitHubhttps://github.com//issues/134#issuecomment-23635901
.

@ctb
Copy link
Member Author

ctb commented Sep 2, 2013

OK. We can leave ec.py in then ;)

@fishjord
Copy link
Contributor

fishjord commented Sep 2, 2013

All for moving it to sandbox though!

On Sun, Sep 1, 2013 at 9:57 PM, C. Titus Brown notifications@github.comwrote:

OK. We can leave ec.py in then ;)


Reply to this email directly or view it on GitHubhttps://github.com//issues/134#issuecomment-23637826
.

@ctb
Copy link
Member Author

ctb commented Sep 3, 2013

done.

On Sun, Sep 01, 2013 at 06:45:18PM -0700, fishjord wrote:

As a reference of what the EC strategy is kinda suppose to work like.
Ideally instead of convoluted logic about whether gaps are in the
beginning or end of the alignment I'm planning on using the truncated
alignment flag that the new read aligner returns as part of the Alignment
object

On Sun, Sep 1, 2013 at 8:01 PM, C. Titus Brown notifications@github.comwrote:

Actually, let's leave ec.py -- it should just be moved to sandbox/ or
tested/documented, I think. @fishjord https://github.com/fishjord, are
you using ec.py?

???
Reply to this email directly or view it on GitHubhttps://github.com//issues/134#issuecomment-23635901
.


Reply to this email directly or view it on GitHub:
#134 (comment)

C. Titus Brown, ctb@msu.edu

@mr-c
Copy link
Contributor

mr-c commented Sep 4, 2013

Has this been merged yet?

@ctb
Copy link
Member Author

ctb commented Sep 4, 2013

No; see #149

@mr-c
Copy link
Contributor

mr-c commented Sep 4, 2013

Done :-)

@mr-c mr-c closed this as completed Sep 4, 2013
ctb added a commit that referenced this issue Apr 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants