snipsco / snips-nlu-rs

Snips NLU rust implementation
https://snips.ai
Other
340 stars 56 forks source link

NLU Configuration filename should not be hardcoded #14

Closed adrienball closed 6 years ago

adrienball commented 6 years ago

Description At the moment, FileBasedConfiguration and ZipBasedConfiguration construction logic relies on an hardcoded filename (which is trained_assistant.json). This seems wrong in terms of testing, code clarity and future-proof-ness.

Instead, these objects should be initialized by passing an additional filename parameter.