TODO - Guy fix random

leondavi · Jul 20, 2024 · c2c2f0f · c2c2f0f
1 parent 1dd4250
commit c2c2f0f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src_erl/NerlnetApp/src/Source/sourceSendingPolicies.erl b/src_erl/NerlnetApp/src/Source/sourceSendingPolicies.erl
@@ -97,7 +97,8 @@ send_method_random(TransmitterEts, Epochs, TimeInterval_ms, ClientWorkerPairs, B
   TotalNumOfBatches = length(BatchesListToSend),
   BatchesIndexes = generate_batch_indexes(TotalNumOfBatches, EpochIdx),
   ClientWorkerPairsLength = length(ClientWorkerPairs),
-  WorkerIdxBatchIdxTuples = [ {X div ClientWorkerPairsLength, rand:uniform(ClientWorkerPairsLength)} || X <- BatchesIndexes],
+  % TODO Guy - there is an issue here  the batch Index should be accumulated according to random selections. 
+  WorkerIdxBatchIdxTuples = [ {X, rand:uniform(ClientWorkerPairsLength)} || X <- BatchesIndexes],
   BatchesWithIndexes = lists:zip(WorkerIdxBatchIdxTuples, BatchesListToSend), % Tuple {{BatchIdx, WorkerIdx}, Batch}
   lists:foreach(BatchFunc, BatchesWithIndexes),
   % update batches sent