RasaHQ / rasa

💬 Open source machine learning framework to automate text- and voice-based conversations: NLU, dialogue management, connect to Slack, Facebook, and more - Create chatbots and voice assistants
https://rasa.com/docs/rasa/
Apache License 2.0
18.91k stars 4.63k forks source link

`rasa data migrate` ignores `out` parameter #10163

Closed ka-bu closed 2 years ago

ka-bu commented 2 years ago

Rasa Open Source version

3.0.0rc

Rasa SDK version

No response

Rasa X version

No response

Python version

3.7

What operating system are you using?

Windows

What happened?

d-r---         8/16/2021  12:09 AM                Searches                                                              
d-----         8/16/2021   8:57 AM                source                                                               
 d-----        11/10/2021   6:51 AM                test1-other                                                           
d-r---         8/16/2021  12:08 AM                Videos                                                               
 -a----        11/10/2021   5:15 AM             25 .condarc                                                                                                                                                                                                                                                                                                              
(rasa) PS C:\Users\User> ls .\test1-other\                                                                              
(rasa) PS C:\Users\User> rasa data migrate  -d .\Documents\test1\domain\ --out .\test1-other\blabla.yml                 
C:\Users\User\anaconda3\envs\rasa\lib\site-packages\rasa\shared\utils\io.py:99: UserWarning: The out path provided is not a directory, creating a new directory 'Documents\test1\new_domain' for migrated domain files.                         Your domain was successfully migrated! It's location is Documents\test1\new_domain. 

Command / Request

No response

Relevant log output

No response

ancalita commented 2 years ago

@ka-bu that message is actually intended because you've provided a directory as a domain and that cannot be migrated to a single yaml file, that's why the file is ignored and a new directory is created. The rasa data migrate needs an existing directory for out parameter if you've provided a directory as domain input. Let me know if that didn't work.

ka-bu commented 2 years ago

works - and there are also warnings that I had overlooked before 👍