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

rasa data convert config does not create data/rules.ymlfile #7113

Closed
koaning opened this issue Oct 27, 2020 · 1 comment · Fixed by #7984
Closed

rasa data convert config does not create data/rules.ymlfile #7113

koaning opened this issue Oct 27, 2020 · 1 comment · Fixed by #7984
Assignees
Labels
area:rasa-oss/cli Issues focused on the rasa command-line-interface area:rasa-oss 🎡 Anything related to the open source Rasa framework area:rasa-oss/training-data Issues focused around Rasa training data (stories, NLU, domain, etc.) effort:atom-squad/2 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. feature:ux-cli+training-data Feature: Improve user experience with Rasa CLI and training data for developers type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@koaning
Copy link
Contributor

koaning commented Oct 27, 2020

I'm working on another season of breakoutbot and I'm converting my project from Rasa 1.10 to Rasa 2.0. Here's the version I've got installed locally;

> rasa --version
Rasa Version     : 2.0.2
Rasa SDK Version : 2.0.0
Rasa X Version   : 0.32.0
Python Version   : 3.7.7
Operating System : Darwin-19.3.0-x86_64-i386-64bit
Python Path      : /Users/vincent/Development/breakoutbot/venv/bin/python3.7

I'm now about to convert a config file that looks like this:

language: en

pipeline:
  - name: WhitespaceTokenizer
  - name: RegexFeaturizer
  - name: LexicalSyntacticFeaturizer
  - name: CountVectorsFeaturizer
  - name: CountVectorsFeaturizer
    analyzer: char_wb
    min_ngram: 1
    max_ngram: 4
  - name: DIETClassifier
    epochs: 100
  - name: EntitySynonymMapper
  - name: ResponseSelector
    epochs: 100

policies:
  - name: MemoizationPolicy
  - name: TEDPolicy
    max_history: 5
    epochs: 100
  - name: MappingPolicy
  - name: FormPolicy

To achieve this I'm running:

> rasa data convert config
Your model configuration contains the 'FormPolicy'. Note that this command does not migrate the 'FormPolicy' and you have to migrate the 'FormPolicy' manually. Please see the migration guide for further details: https://rasa.com/docs/rasa/migration-guide
2020-10-27 11:37:09 INFO     rasa.shared.utils.validation  - The 'version' key is missing in the training data file /Users/vincent/Development/breakoutbot/domain.yml. Rasa Open Source will read the file as a version '2.0' file. See https://rasa.com/docs/rasa/training-data-format.
Output file 'data/rules.yml' did not exist and will be created.
Migrating the 'MappingPolicy'.
Finished migrating your policy configuration 🎉.
The migration generated 0 rules which were added to 'data/rules.yml'.

The feedback I get suggests that a new file has been created over at data/rules.yml. Unfortunately, this does not seem to be the case.

> ls data 
total 16
-rw-r--r--   1 vincent  staff   1.1K 27 Oct 11:17 stories.yml
drwxr-xr-x   4 vincent  staff   128B 27 Oct 11:17 .
-rw-r--r--   1 vincent  staff   2.9K 27 Oct 11:17 nlu.yml
drwxr-xr-x  30 vincent  staff   960B 20 Oct 13:12 ..

It's probably due to the fact that zero rules needed to be generated. This feels like a bug though, the user feedback does not match what is happening to my filesystem.

@koaning koaning added type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. area:rasa-oss 🎡 Anything related to the open source Rasa framework labels Oct 27, 2020
@koaning
Copy link
Contributor Author

koaning commented Oct 27, 2020

It seems like some more unexpected behavior is happening. When running the convert command it seems like the config.yaml file does not change despite the logs mentioning Migrating the 'MappingPolicy'.

@wochinge wochinge added area:rasa-oss/training-data Issues focused around Rasa training data (stories, NLU, domain, etc.) priority:high area:rasa-oss/cli Issues focused on the rasa command-line-interface labels Jan 29, 2021
@wochinge wochinge added the effort:atom-squad/2 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. label Feb 12, 2021
@TyDunn TyDunn added the feature:ux-cli+training-data Feature: Improve user experience with Rasa CLI and training data for developers label Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-oss/cli Issues focused on the rasa command-line-interface area:rasa-oss 🎡 Anything related to the open source Rasa framework area:rasa-oss/training-data Issues focused around Rasa training data (stories, NLU, domain, etc.) effort:atom-squad/2 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. feature:ux-cli+training-data Feature: Improve user experience with Rasa CLI and training data for developers type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
4 participants