Quality Critieria are listed twice in the provenance log

[kosack]

Describe the bug

Both the QualityQuery and its internal ExpressionEngine's criteria get written to the provenance log. It seems both return their configuration, but this is redundant info. This also means both are in principle configurable in a config file, which could be confusing: which gets precedence in case of both being used?

This is likely because ExpressionEngine inherits from Component: I don't think it should, it would be fine as a bare class since it is not a processing component.

Example:

              "ImageQualityQuery": {
                  "quality_criteria": [
                     [
                        "size_greater_0",
                        "image.sum() > 0"
                     ]
                  ],
                  "ExpressionEngine": {
                     "expressions": [
                        [
                           "size_greater_0",
                           "image.sum() > 0"
                        ]
                     ]
                  }
              }

To Reproduce

Check the provenance.log after running ctapipe-process.

Expected behavior

Only the configurable criteria should appear.