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

Add basic flow support #12134

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Add basic flow support #12134

wants to merge 3 commits into from

Conversation

znat
Copy link
Contributor

@znat znat commented Mar 13, 2023

Flows as a loop

  • Linear flows only (no conditions)
  • Mappings work
  • Slot validation doesn't work (requires a change in the sdk)

Project repo: https://github.com/RasaHQ/rasa-project-with-flows

@@ -1020,6 +1028,18 @@ def __init__(self) -> None:
super().__init__("utter_ask_rephrase", silent_fail=True)


class ActionExtractSteps(Action):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's this used for? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dont think it is :)
This whole PR is just crap code not meant to be reviewed yet :) Just messing around

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha yeah for sure and i know, just trying to understand what you did :)

logger = logging.getLogger(__name__)


class FlowAction(FormAction):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

before we go and implement more in this direction we should check if an action actually is the right place.

my first guess would have been that this should be a policy (like the rule policy) tbh - but I am not so sure about this. let me open a thread in the engine channel to check what others think

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(might also something that we need to toy with)

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

Successfully merging this pull request may close these issues.

2 participants