Add migration guide for policies #7522
Conversation
|
|
||
| ### Policies | ||
|
|
||
| [Policies](./policies.mdx) now require a `**kwargs` argument in their constructor and `load` method. |
There was a problem hiding this comment.
we also have a changelog for this, right?
There was a problem hiding this comment.
hmm I don't think so. Will we have one changelog for the whole of continuous training?
There was a problem hiding this comment.
we should also add a changelog mentioning this change and then e.g. point to the migration guide.
Will we have one changelog for the whole of continuous training?
I think this change is somewhat unrelated to the training (we needed it, but we could have also done the policy interface change on its own) so I think it deserves it's own changelog. Especially because it's a deprecation.
There was a problem hiding this comment.
ok I can add it in this PR then. What issue number should we use?
There was a problem hiding this comment.
mhm, either the number of this PR or probably better the one from the PR which introduced the change.
Proposed changes:
Add migration steps for rasa 2.1 to 2.2 caused by continuous training
Status (please check what you already did):
black(please check Readme for instructions)