Skip to content

count_reads.py fix when library could be a tsv - #6

Open
SimonLammmm wants to merge 47 commits into
johncthomas:masterfrom
SimonLammmm:master
Open

count_reads.py fix when library could be a tsv#6
SimonLammmm wants to merge 47 commits into
johncthomas:masterfrom
SimonLammmm:master

Conversation

@SimonLammmm

Copy link
Copy Markdown
Contributor

In count_reads.py:576, pd.read_table() in favour over pd.read_csv() so as to allow if the library is a tsv

…les the file downstream decide what to do, and don't let the function return None
… with `--overwrite`

In a previous commit, the behaviour that the pipeline skips over existing analysis files was removed, but existing analysis files were still checked for. I have re-enabled the behaviour and made it the default. This is so that additional analyses can be easily added without running everything again. Overwriting existing analysis files can be achieved with `--overwrite`.
@SimonLammmm

Copy link
Copy Markdown
Contributor Author

All planned changes have been committed and I have completed my tests. This pull request is ready to merge :)

Improve Chronos creation of sequencemap
Chronos deal properly when replicates are used in more than one comparison
Add Trajectory column in replicate details xlsx for Chronos analysis
If supplied, detect guide lengths. If guide length is not constant or slice width is not equal to guide length, slide a window along the slice and count individually. Retain support when library not supplied
Support variable guides and mismatched slice in count_reads
Warn instead of fail when using a slice shorter than max guide length in count_reads
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

Successfully merging this pull request may close these issues.

1 participant