Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Training/Testing Number Via Cli? #84

Open
ttlekich opened this issue Dec 6, 2019 · 2 comments
Open

Training/Testing Number Via Cli? #84

ttlekich opened this issue Dec 6, 2019 · 2 comments

Comments

@ttlekich
Copy link

ttlekich commented Dec 6, 2019

Is there an option to dynamically pass the number of training and testing examples?

For example, say I have a chatito files that looks like this:

%[intent_name_1]('training': '100')
...

%[intent_name_2]('training': '100')
...

but I would like to have files that look like this:

%[intent_name_1]
...

%[intent_name_2]
...

and specify the training set size (same number for all intents or an option object to pass for each individual intent) when I invoke chatito:

npx chatito ... --training=100

Of course, I could modify the chatito files dynamically with the desired options before I run chatito, but thought it might be nice if there is a simple option like I described.

I could not find my answer in the docs or previous issues, but I apologize in advance if I missed something obvious! Thanks!

@rodrigopivi
Copy link
Owner

rodrigopivi commented Dec 6, 2019

Hi @ttlekich,

Thanks for the suggestion, that is not implemented, currently, for the example:

%[intent_name_1]
...

The CLI generates all unique combinations. I agree that it can be useful to set some of those argument level values dynamically (¿maybe considering being able to reference ENV/outside/cli-level variables from templates?), something like:

%[intent_name_1]('training': '$TRAINING', 'testing': '$TRAINING')
...

This could be an option. And would allow sending params as arguments/env variables?
Adding support for referencing custom variables from the template might be a useful addition for the future, although that is not something I'm planning to implement in the near future, feel free to open a PR if you like. Thanks

@ttlekich
Copy link
Author

ttlekich commented Dec 9, 2019

@rodrigopivi thank you for the reply! I will certainly take a look at implementing the environment variable injection when I get the time (and open a PR). Also thank you for a fantastic project - I've enjoyed using chatito.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants