Merge 27c5b37 into 05407a2

CHANGELOG.rst

@@ -12,6 +12,7 @@ Added
 
 Changed
 -------
+- validate training data only if used for training
 - applied spacy guidelines on how to disable pipeline components
 
 Removed

rasa_nlu/model.py

@@ -163,6 +163,8 @@ def train(self, data: TrainingData, **kwargs: Any)-> 'Interpreter':
 
         self.training_data = data
 
+        self.training_data.validate()
+
         context = kwargs
 
         for component in self.pipeline:

rasa_nlu/training_data/loading.py

@@ -13,6 +13,7 @@
 
 if typing.TYPE_CHECKING:
     from rasa_nlu.training_data import TrainingData
+    from rasa_nlu.training_data.formats.readerwriter import TrainingDataReader
 
 logger = logging.getLogger(__name__)
 
@@ -56,7 +57,6 @@ def load_data(resource_name: Text,
     else:
         training_data = data_sets[0].merge(*data_sets[1:])
 
-    training_data.validate()
     return training_data
 
 
@@ -72,15 +72,14 @@ def load_data_from_endpoint(data_endpoint: EndpointConfig,
         temp_data_file = utils.create_temporary_file(response.content,
                                                      mode="w+b")
         training_data = _load(temp_data_file, language)
-        training_data.validate()
 
         return training_data
     except Exception as e:
         logger.warning("Could not retrieve training data "
                        "from URL:\n{}".format(e))
 
 
-def _reader_factory(fformat):
+def _reader_factory(fformat: Text) -> Optional['TrainingDataReader']:
     """Generates the appropriate reader class based on the file format."""
     from rasa_nlu.training_data.formats import (
         MarkdownReader, WitReader, LuisReader,
@@ -100,7 +99,8 @@ def _reader_factory(fformat):
     return reader
 
 
-def _load(filename, language='en'):
+def _load(filename: Text, language: Optional[Text] = 'en'
+          ) -> Optional['TrainingData']:
     """Loads a single training data file from disk."""
 
     fformat = _guess_format(filename)