sebwrede commented on a change in pull request #1395:
URL: https://github.com/apache/systemds/pull/1395#discussion_r723401719
##########
File path: src/main/java/org/apache/sysds/hops/cost/FederatedCostEstimator.java
##########
@@ -43,7 +43,7 @@
public int DEFAULT_MEMORY_ESTIMATE = 8;
public int DEFAULT_ITERATION_NUMBER = 15;
public double WORKER_NETWORK_BANDWIDTH_BYTES_PS = 1024*1024*1024;
//Default network bandwidth in bytes per second
- public double WORKER_COMPUTE_BANDWITH_FLOPS = 2.5*1024*1024*1024;
//Default compute bandwidth in FLOPS
+ public double WORKER_COMPUTE_BANDWIDTH_FLOPS = 2.5*1024*1024*1024;
//Default compute bandwidth in FLOPS
Review comment:
I know, but in this case I thought it looked better like this because it
is a constant being initialized by constants, so it is a special case.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
