Closed dnguyen1196 closed 4 years ago
This pull request introduces 1 alert when merging 6f046e6f1b483afa2839dbaeb07f90665401bcf9 into 31fe04671132717b0fbbebabbefe07fdaed81079 - view on LGTM.com
new alerts:
This pull request introduces 1 alert when merging 39519c79952649a3a7739690209133b88f67c3cc into 53cb5b2c6f6c280f2ec84d0c69065b45a4c72200 - view on LGTM.com
new alerts:
Allows for mixing different convolution layer types. Implementation can be found in
pinot/representation/sequential.py
. However, specifying the config is not a little more verbose and has to follow specific order as a tradeoff for this increased flexibility. The config has to be specified like this:And the order of the parameters have to follow the same order as specified in
pinot/representation/sequential.layer_param_type
Users can specify config as a list of string, which makes it able to handle command-line output. The code will automatically cast the parameter values to the right type. For example