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
Instrumentation for RegressPhenotypes #18
Conversation
Can one of the admins verify this patch? |
I was waiting on a successful build for #12 before adding other pull requests to Jenkins. |
// Log the results | ||
logResults(associations, sc) | ||
} | ||
|
||
def print_metrics_to_cmdline(metricsListener: MetricsListener) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use camelCase for method names
@@ -95,6 +97,9 @@ class RegressPhenotypesArgs extends Args4jBase { | |||
|
|||
@Args4jOption(required = false, name = "-oneTwo", usage = "If cases are 1 and controls 2 instead of 0 and 1") | |||
var oneTwo = false | |||
|
|||
@Args4jOption(required = false, name = "-time", usage = "Prints out the timing information for RegressPhenotypes to commandline.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in ADAM we use -print_metrics
for this, which comes from bdg-utils
Or am I missing the use case here?
@@ -107,6 +112,11 @@ class RegressPhenotypes(protected val args: RegressPhenotypesArgs) extends BDGSp | |||
val a = args.oneTwo | |||
// println(s"\n\n\n\n\n\n oneTwo: $a \n\n\n\n\n\n\n\n") | |||
|
|||
import Timers._ | |||
Metrics.initialize(sc) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is also handled in bdg-utils for the -print_metrics
argument
Jenkins, add to whitelist |
Ports in incremental update refactor code
Adds in instrumentation for RegressPhenotypes which can be ported to other section of code when the need arises.