lanking520 commented on a change in pull request #13995: [MXNET-1287][WIP]
Scala compiler warnings
URL: https://github.com/apache/incubator-mxnet/pull/13995#discussion_r251551640
##########
File path:
scala-package/core/src/main/scala/org/apache/mxnet/ExecutorManager.scala
##########
@@ -395,17 +395,17 @@ private[mxnet] object ExecutorManager {
* @param paramNames Names of all trainable parameters.
* @param ctx List of devices for training (data parallel)
* @param slices Describes how the data parallel splits data into different
devices.
- * @param providedData training data shapes
- * @param providedLabel training label shapes
+ * @param providedDataDesc training data descriptions
+ * @param providedLabelDesc training label descriptions
* @param sharedGroup: DataParallelExecutorGroup
* An existing executor group, if to share parameters with
it.
*
*/
private class DataParallelExecutorGroup private(sym: Symbol,
argNames: IndexedSeq[String], paramNames:
Set[String],
ctx: Array[Context], private val slices:
Array[(Int, Int)],
- providedData: Map[String, Shape],
- providedLabel: Map[String, Shape],
+ providedDataDesc: IndexedSeq[DataDesc],
Review comment:
Why we are making this changes?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
