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

Support custom importers in rasa data validate command #11448

Closed
kozlice opened this issue Aug 13, 2022 · 1 comment · Fixed by #11470
Closed

Support custom importers in rasa data validate command #11448

kozlice opened this issue Aug 13, 2022 · 1 comment · Fixed by #11470
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR

Comments

@kozlice
Copy link
Contributor

kozlice commented Aug 13, 2022

What problem are you trying to solve?

Custom importers are only used in training currently. It would be great to use them during validation as well.

What's your suggested solution?

#11470

Examples (if relevant)

No response

Is anything blocking this from being implemented? (if relevant)

No response

Definition of Done

rasa data validate uses custom importers if provided in config.

@kozlice kozlice added area:rasa-oss 🎡 Anything related to the open source Rasa framework type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR labels Aug 13, 2022
@ancalita ancalita linked a pull request Aug 24, 2022 that will close this issue
4 tasks
@ancalita
Copy link
Member

ancalita commented Sep 8, 2022

Issue done.

@ancalita ancalita closed this as completed Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants