Skip to content

Commit

Permalink
#328: TonY shouldn't throw GPUDiscoverer errors when GPUs are not req…
Browse files Browse the repository at this point in the history
…uested (#329)
  • Loading branch information
UWFrankGu authored and erwa committed Jun 17, 2019
1 parent 42db0c6 commit 1d8f077
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ ext.deps = [

allprojects {
group = "com.linkedin.tony"
project.version = "0.3.15"
project.version = "0.3.16"
}

task sourcesJar(type: Jar) {
Expand Down
4 changes: 2 additions & 2 deletions tony-core/src/main/java/com/linkedin/tony/TaskMonitor.java
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,8 @@ void initMetrics() {

private boolean checkIsGpuMachine(Configuration conf) {
int numWorkerGpus = conf.getInt(
TonyConfigurationKeys.getResourceKey("worker", "gpus"), 0);
LOG.info("Number of GPUs requested: " + numWorkerGpus);
TonyConfigurationKeys.getResourceKey(taskType, "gpus"), 0);
LOG.info("Number of GPUs requested for " + taskType + ": " + numWorkerGpus);
return numWorkerGpus > 0;
}

Expand Down

0 comments on commit 1d8f077

Please sign in to comment.