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

Remove PairingRDD #1122

Closed
fnothaft opened this Issue Aug 23, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@fnothaft
Member

fnothaft commented Aug 23, 2016

After #1108 merges, PairingRDD will be unused and should be removed.

@fnothaft fnothaft self-assigned this Aug 23, 2016

fnothaft added a commit to fnothaft/adam that referenced this issue Aug 23, 2016

akmorrow13 added a commit to akmorrow13/adam that referenced this issue Aug 23, 2016

Merge pull request #1 from fnothaft/issues/1122-remove-pairing-rdd
[ADAM-1122] Remove Coverage and PairingRDD. Resolves #1122.

akmorrow13 added a commit to akmorrow13/adam that referenced this issue Aug 23, 2016

akmorrow13 added a commit to akmorrow13/adam that referenced this issue Aug 23, 2016

refactored ReferenceFile to require SequenceDictionary
Added support for coverage calculation from alignment files

[ADAM-1122] Remove Coverage and PairingRDD. Resolves #1122.
@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 24, 2016

Member

Is it still useful, more generally? bigdatagenomics/utils#16

Member

heuermh commented Aug 24, 2016

Is it still useful, more generally? bigdatagenomics/utils#16

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Aug 24, 2016

Member

IMO, no.

Member

fnothaft commented Aug 24, 2016

IMO, no.

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 24, 2016

Member

ok, when this issue closes, also close the utils issue linked above as WontFix.

Member

heuermh commented Aug 24, 2016

ok, when this issue closes, also close the utils issue linked above as WontFix.

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Aug 24, 2016

Member

SGTM!

Member

fnothaft commented Aug 24, 2016

SGTM!

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