Misc. INDEL realigner bugfixes #1382

Merged
merged 3 commits into from Feb 21, 2017

Conversation

Projects
None yet
3 participants
@fnothaft
Member

fnothaft commented Feb 5, 2017

No description provided.

@fnothaft fnothaft added this to the 0.21.1 milestone Feb 5, 2017

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Feb 5, 2017

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1773/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1773/
Test PASSed.

+ new RichAlignmentRecord(builder.build())
+ } catch {
+ case t: Throwable => {
+ println("Caught %s when trying to move alignment to %s for %s.".format(

This comment has been minimized.

@heuermh

heuermh Feb 6, 2017

Member

use Logger here?

@heuermh

heuermh Feb 6, 2017

Member

use Logger here?

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Feb 21, 2017

Member

@fnothaft ping for logging fix

Member

heuermh commented Feb 21, 2017

@fnothaft ping for logging fix

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Feb 21, 2017

Member

Fixed logging and rebased.

Member

fnothaft commented Feb 21, 2017

Fixed logging and rebased.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Feb 21, 2017

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1812/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1812/
Test PASSed.

@heuermh heuermh merged commit a6b2cdf into bigdatagenomics:master Feb 21, 2017

1 check passed

default Merged build finished.
Details
@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Feb 21, 2017

Member

Thank you, @fnothaft

Member

heuermh commented Feb 21, 2017

Thank you, @fnothaft

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