-
Notifications
You must be signed in to change notification settings - Fork 46
Maybe a bug in CNN #31
Copy link
Copy link
Open
Description
Hello,
I was reading the code of the CNN model and noticed an issue in the AdaptiveConcatPool1d class. Both pooling methods are currently using AdaptiveAvgPool1d, shouldn't it be avg and max?
https://github.com/TheDatumOrg/TSB-AD/blob/dce9e5e5ec14ffb82787315dbf1a8564c7bb9f47/TSB_AD/models/CNN.py#L13-L18
Thank you!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels