Custom paths for corpus data

[vkosuri]

I think it would be good option to specify custom paths for corpus data, currently its system specified path only

class Corpus(object):

    def __init__(self):
        current_directory = os.path.dirname(__file__)
        self.data_directory = os.path.join(current_directory, 'data')

[vkosuri]

@gunthercox Planning to write like this, any suggestions/comments?

@@ -9,6 +9,7 @@ class Trainer(object):
 
     def __init__(self, storage, **kwargs):
         self.storage = storage
+        self.corpus_path = None
         self.logger = logging.getLogger(__name__)
 
     def train(self, *args, **kwargs):
@@ -54,6 +55,10 @@ class Trainer(object):
 
         return result
 
+    def import_for_training(self, file_path):
+        self.corpus_path = file_path
+
+
     def export_for_training(self, file_path='./export.json'):
         """
         Create a file from the database that can be used to
@@ -100,7 +105,7 @@ class ChatterBotCorpusTrainer(Trainer):
         super(ChatterBotCorpusTrainer, self).__init__(storage, **kwargs)
         from .corpus import Corpus
 
-        self.corpus = Corpus()
+        self.corpus = Corpus(self.corpus_path)
 
     def train(self, *corpora):
         trainer = ListTrainer(self.storage)

[gunthercox]

@vkosuri Looks good to me 👍

[rajasimon]

I like this to be integrated asap!

[vkosuri]

Sure 👍

…

On 21-Dec-2016 12:18, "Raja Simon" ***@***.***> wrote: I like this to be integrated asap! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#467 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ANCAAcCoRG0Rs25sLHEnQ1s10oB_fRpaks5rKMvbgaJpZM4LEHSY> .

[rajasimon]

Can you make the pull request so I can check it and also does it support for adding my own custom adapters...?

i.e)

    'logic_adapters': [
        'chatterbot.logic.MathematicalEvaluation',
        'chatterbot.logic.BestMatch'
        'funstuff.logic.FunStuff'
    ],

[rajasimon]

Sorry for my prev message I just saw this https://chatterbot.readthedocs.io/en/stable/logic/create-a-logic-adapter.html#directory-structure

[rajasimon]

@vkosuri Shall I have the patch now ?

[vkosuri]

Sure