Merge 189f9a4 into 2ecaa5b

CHANGELOG.rst

@@ -7,14 +7,22 @@ Rasa Change Log
 All notable changes to this project will be documented in this file.
 This project adheres to `Semantic Versioning`_ starting with version 1.0.
 
+
+[1.3.4] - Unreleased
+^^^^^^^^^^^^^^^^^^^^
+
+Fixed
+-----
+- Policy ensemble no longer incorrectly wrings "missing mapping policy" when
+  mapping policy is present.
+
 [1.3.3] - 2019-09-11
 ^^^^^^^^^^^^^^^^^^^^
 
 Fixed
 -----
 - Added a check to avoid training CountVectorizer for a particular attribute of a message if no text is provided for that attribute across the training data.
 
-
 [1.3.2] - 2019-09-10
 ^^^^^^^^^^^^^^^^^^^^
 

rasa/core/policies/ensemble.py

@@ -25,6 +25,7 @@
 from rasa.core.exceptions import UnsupportedDialogueModelError
 from rasa.core.featurizers import MaxHistoryTrackerFeaturizer
 from rasa.core.policies.policy import Policy
+from rasa.core.policies.mapping_policy import MappingPolicy
 from rasa.core.policies.fallback import FallbackPolicy
 from rasa.core.policies.memoization import MemoizationPolicy, AugmentedMemoizationPolicy
 from rasa.core.trackers import DialogueStateTracker
@@ -53,7 +54,7 @@ def __init__(
         self._check_for_important_policies()
 
     def _check_for_important_policies(self):
-        if "MappingPolicy" not in self.policies:
+        if not any([isinstance(policy, MappingPolicy) for policy in self.policies]):
             logger.info(
                 "MappingPolicy not included in policy ensemble. Default intents "
                 "'{} and {} will not trigger actions '{}' and '{}'."