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 bbtools version and parameters #4

Closed
aclum opened this issue May 9, 2024 · 11 comments
Closed

update bbtools version and parameters #4

aclum opened this issue May 9, 2024 · 11 comments
Assignees

Comments

@aclum
Copy link

aclum commented May 9, 2024

JGI is on bbtools 39.03

Alicia to confirm final parameters with Brian.

cc @kaijli

@aclum aclum assigned aclum and kaijli and unassigned aclum May 9, 2024
@ssarrafan ssarrafan added the backlog backlogged to be reprioritized in the future label May 20, 2024
@ssarrafan
Copy link

Sprint over, removing from sprint.

@aclum aclum assigned aclum and kaijli and unassigned kaijli and aclum May 20, 2024
@kaijli
Copy link
Collaborator

kaijli commented Jun 13, 2024

@aclum is it confirmed? is there a specific docker image for that?

@aclum aclum assigned aclum and unassigned kaijli Jun 15, 2024
@aclum aclum removed the backlog backlogged to be reprioritized in the future label Jun 15, 2024
@aclum
Copy link
Author

aclum commented Jun 15, 2024

I'll finalize this next week. There were some recommendations for removing optical duplicates that I need to review.

@aclum
Copy link
Author

aclum commented Jun 24, 2024

The instrument name in the schema currently is a string field so we can't automate removal of optical duplicates settings. We'll revisit this once Berkeley schema is out in ~October.

@kaijli Please update the bbtools container 39.03.

@aclum aclum assigned kaijli and unassigned aclum Jun 24, 2024
@kaijli
Copy link
Collaborator

kaijli commented Jun 25, 2024

bumped bbtools ver.
successful run at jaws id 79956
dd222dc

@ssarrafan
Copy link

Will move to next sprint for review
@aclum @kaijli

@aclum
Copy link
Author

aclum commented Jul 1, 2024

Over slack based on a conversation with Brian we decided to also add extend=f, this was being turned on behind the scenes which was enabling tadpole and using up large amounts of memory.
If we enable extend=t we should be able to reduce the memory for the rqc filter task to 115 Gb, leave threads at 64 and CPU at 32.

@kaijli
Copy link
Collaborator

kaijli commented Jul 1, 2024

successful run with added extend=f and lowered memory at jaws id 80306

@aclum
Copy link
Author

aclum commented Jul 2, 2024

Looks great. Last thing is we need to add verifypaired=t to stage_interleave

@kaijli
Copy link
Collaborator

kaijli commented Jul 4, 2024

17ffa95 done, successful run on expanse

@ssarrafan
Copy link

@kaijli @aclum based on the last comment from Kaitlyn assuming this is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants