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

Removing ProgramRecords info when saving data to sam/bam? #1257

Closed
liamlee opened this Issue Nov 10, 2016 · 4 comments

Comments

Projects
3 participants
@liamlee
Contributor

liamlee commented Nov 10, 2016

Hi,

Before ADAM,I got PG info in the sam file:
image

After ADAM, the output sam/bam file lost the PG info
image

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Nov 10, 2016

Member

Agreed, this is something we are missing. I've opened bigdatagenomics/bdg-formats#114 as we'll need an upstream schema addition. I think this implementation will be similar to #1233.

Member

fnothaft commented Nov 10, 2016

Agreed, this is something we are missing. I've opened bigdatagenomics/bdg-formats#114 as we'll need an upstream schema addition. I think this implementation will be similar to #1233.

@fnothaft fnothaft added this to the 0.22.0 milestone Nov 10, 2016

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Nov 10, 2016

Member

I don't think this will make the upcoming 0.21.0 release, but we'll pick this up in 0.22.0.

Member

fnothaft commented Nov 10, 2016

I don't think this will make the upcoming 0.21.0 release, but we'll pick this up in 0.22.0.

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Nov 10, 2016

Member

+1

Member

heuermh commented Nov 10, 2016

+1

@liamlee

This comment has been minimized.

Show comment
Hide comment
@liamlee

liamlee Nov 12, 2016

Contributor

Looking forward to this .

Contributor

liamlee commented Nov 12, 2016

Looking forward to this .

@heuermh heuermh added this to Triage in Release 0.23.0 Mar 8, 2017

@heuermh heuermh moved this from Triage to Completed in Release 0.23.0 Jan 4, 2018

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