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

bumped version, added notes to CHANGES #22

Closed
wants to merge 10 commits into from
Closed

bumped version, added notes to CHANGES #22

wants to merge 10 commits into from

Conversation

byb121
Copy link
Contributor

@byb121 byb121 commented Jul 11, 2019

No description provided.

@byb121 byb121 requested a review from keiranmraine July 11, 2019 15:40
Copy link
Contributor

@keiranmraine keiranmraine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. The version in the Docker file needs to match the release:

version="1.0.0" \

  1. Not a blocker but, is there any artificial limit of CPUs used in the tophat_fusion.pl script? Pipeline version is requesting 16 cpu, but load is only ~2, looking at point where tophat-fusion-post is executing under this there are only 5 processes.

@byb121
Copy link
Contributor Author

byb121 commented Jul 12, 2019

need to solve: #23

@byb121 byb121 closed this Aug 1, 2019
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.

2 participants