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

Update README with gene collapsing and insert size commands. #18

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

roryk
Copy link

@roryk roryk commented Aug 17, 2017

Thanks, this pull request does two things. It updates the documentation to add the last step to collapse down to the gene level fusion calls. It also renames the two python scripts to be prepended with pizzly-- those scripts are pretty generically named so they might introduce some clashes if we install them as binaries in the conda package. Prepending them with pizzly will get around that. Thanks!

This also renames the two helper scripts to be prefixed with pizzly. This way
we can load them as binaries with bioconda which will help with incorporation
as part of analysis pipelines.

Closes pmelsted#17.
@roryk
Copy link
Author

roryk commented Sep 5, 2017

I fixed the bioconda recipe to install the two python scripts as pizzly_ to match this pull request.

@roryk
Copy link
Author

roryk commented Sep 6, 2017

I added the shebang to the top of the python scripts which will make them executable via the users default python without prepending with python.

@roryk
Copy link
Author

roryk commented Sep 6, 2017

I also fixed a bug in the fragment length script, where the cutoff was set as the default even if a cutoff was supplied.

@roryk
Copy link
Author

roryk commented Oct 16, 2017

ping!

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