kotlindl icon indicating copy to clipboard operation
kotlindl copied to clipboard

Add input/outputShape information to all KDocs of layers (#211)

Open dosier opened this issue 3 years ago • 1 comments

dosier avatar Sep 29 '22 06:09 dosier

Sorry for such a late review. There are a few minor issues with the docs. You make the following comment in many places:

Use the keyword argument 'input_shape'
 (tuple of integers, does not include the samples axis)
 when using this layer as the first layer in a model.

But currently, we only support Input to be the first layer of the network. Moreover, there is no argument named input_shape in the constructor anywhere in the code.

I see, thank you for the feedback! Also my apologies for the late reply, I have been quite busy. Do u have any suggestions on what to replace that with? E.g. "Depends on the Input layer"'? Or can it be omitted?

dosier avatar Mar 02 '23 20:03 dosier