Prohibit additional properties in scope_overrides section of NNCFConf…

…ig (#1117)
openvinotoolkit · Mar 28, 2022 · d375cd0 · d375cd0
1 parent d48d16c
commit d375cd0
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 9 deletions.
diff --git a/docs/compression_algorithms/Quantization.md b/docs/compression_algorithms/Quantization.md
@@ -368,12 +368,14 @@ sparsity and filter pruning algorithms. It can be enabled by setting a non-zero
     },
     "quantize_inputs": true, // Whether the model inputs should be immediately quantized prior to any other model operations."
     "scope_overrides": { // This option is used to specify overriding quantization constraints for specific scope, e.g. in case you need to quantize a single operation differently than the rest of the model.
-        "{re}.*InvertedResidual.*": {
-            "mode": "symmetric", // Mode of quantization
-            "bits": 4, // Bitwidth to quantize to.
-            "signed": true, // Whether to use signed or unsigned input/output values for quantization. If specified as unsigned and the input values during initialization have differing signs, will reset to performing signed quantization instead.
-            "per_channel": false // Whether to quantize inputs per channel (i.e. per 0-th dimension for weight quantization,and per 1-st dimension for activation quantization)
-        }
+    	"activations": {
+		"{re}.*InvertedResidual.*": {
+		    "mode": "symmetric", // Mode of quantization
+		    "bits": 4, // Bitwidth to quantize to.
+		    "signed": true, // Whether to use signed or unsigned input/output values for quantization. If specified as unsigned and the input values during initialization have differing signs, will reset to performing signed quantization instead.
+		    "per_channel": false // Whether to quantize inputs per channel (i.e. per 0-th dimension for weight quantization,and per 1-st dimension for activation quantization)
+		}
+	}
     },
 
     // A list of model control flow graph node scopes to be ignored for this operation - functions as a 'denylist'. Optional.

diff --git a/examples/torch/classification/configs/quantization/rmnet_cifar_int8.json b/examples/torch/classification/configs/quantization/rmnet_cifar_int8.json
@@ -20,9 +20,11 @@
     "compression": {
         "algorithm": "quantization",
         "scope_overrides": {
-            "{re}RMBlock\\[[0-9]*\\]/ELU\\[activation\\]": {
-                "signed": true
-            }
+		"activations": {
+			    "{re}RMBlock\\[[0-9]*\\]/ELU\\[activation\\]": {
+				"signed": true
+			    }
+		}
         }
     }
 }
diff --git a/nncf/config/schema.py b/nncf/config/schema.py
@@ -554,6 +554,7 @@ def with_attributes(schema: Dict, **kwargs) -> Dict:
                     },
                 }
             },
+            "additionalProperties": False,
             "description": "This option is used to specify overriding quantization constraints for specific scope,"
                            "e.g. in case you need to quantize a single operation differently than the rest of the "
                            "model."