Skip to content

Commit

Permalink
[DOC] Remove extra parameter in docstring with incorrect definition (#…
Browse files Browse the repository at this point in the history
…5617)


There are two `optimizer`'s called out in the `CNNClassifier` class's
docstring. One of them is incorrect and references `keras.optimzer` when
it should be `keras.optimizers`.

This change removes the incorrect parameter in the docstring.
  • Loading branch information
wayneadams committed Dec 22, 2023
1 parent d35b470 commit 3a4921a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
9 changes: 9 additions & 0 deletions .all-contributorsrc
Expand Up @@ -2480,6 +2480,15 @@
"code"
]
},
{
"login": "wayneadams",
"name": "Wayne Adams",
"avatar_url": "https://avatars.githubusercontent.com/u/15034841?s=400&u=d717e9945910bcc844c5e64cd56d570c6cc4e8e6&v=4",
"profile": "https://github.com/wayneadams",
"contributions": [
"doc"
]
},
{
"login": "sssilvar",
"name": "Santiago Smith Silva",
Expand Down
1 change: 0 additions & 1 deletion sktime/classification/deep_learning/cnn.py
Expand Up @@ -34,7 +34,6 @@ class CNNClassifier(BaseDeepClassifier):
whether to output extra information
loss : string, default="mean_squared_error"
fit parameter for the keras model
optimizer : keras.optimizer, default=keras.optimizers.Adam(),
metrics : list of strings, default=["accuracy"],
activation : string or a tf callable, default="sigmoid"
Activation function used in the output linear layer.
Expand Down

0 comments on commit 3a4921a

Please sign in to comment.