#movinet set NormActivation.activation = None by default and gauge its use case.

PiperOrigin-RevId: 416126695

official/vision/beta/projects/movinet/configs/movinet.py

@@ -126,7 +126,7 @@ class MovinetModel(video_classification.VideoClassificationModel):
   model_type: str = 'movinet'
   backbone: Backbone3D = Backbone3D()
   norm_activation: common.NormActivation = common.NormActivation(
-      activation='swish',
+      activation=None,  # legacy flag, not used.
       norm_momentum=0.99,
       norm_epsilon=1e-3,
       use_sync_bn=True)

official/vision/beta/projects/movinet/modeling/movinet.py

@@ -709,8 +709,12 @@ def build_movinet(
   """Builds MoViNet backbone from a config."""
   backbone_type = backbone_config.type
   backbone_cfg = backbone_config.get()
-  assert backbone_type == 'movinet', ('Inconsistent backbone type '
-                                      f'{backbone_type}')
+  if backbone_type != 'movinet':
+    raise ValueError(f'Inconsistent backbone type {backbone_type}')
+  if norm_activation_config.activation is not None:
+    raise ValueError(
+        'norm_activation is not used in MoViNets, but specified: %s' %
+        norm_activation_config.activation)
 
   return Movinet(
       model_id=backbone_cfg.model_id,