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

Refactors ASR node to leverage and emit DialogueUtterance #421

Merged
merged 3 commits into from
May 24, 2024

Conversation

derkmed
Copy link
Contributor

@derkmed derkmed commented May 21, 2024

This will allow nodes to be reordered or dropped easily as desired. Furthermore, other nodes can easily be added using the more modular DialogueUtterance msg.

@derkmed derkmed force-pushed the dialogue_refactor_existing_2 branch from 537be90 to 1a9ddc7 Compare May 21, 2024 02:56
@derkmed derkmed changed the title Refactors all dialogue system nodes to leverage DialogueUtterance Refactors ASR nodes to leverage and emit DialogueUtterance May 21, 2024
@derkmed derkmed changed the title Refactors ASR nodes to leverage and emit DialogueUtterance Refactors ASR node to leverage and emit DialogueUtterance May 21, 2024
Purg
Purg previously approved these changes May 23, 2024
@derkmed derkmed dismissed Purg’s stale review May 23, 2024 17:32

The merge-base changed after approval.

Purg
Purg previously approved these changes May 23, 2024
@derkmed derkmed dismissed Purg’s stale review May 23, 2024 18:07

The merge-base changed after approval.

@Purg Purg force-pushed the dialogue_refactor_existing_2 branch from 9a047a5 to b87f81f Compare May 23, 2024 18:07
Copy link
Contributor Author

@derkmed derkmed left a comment

Choose a reason for hiding this comment

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

LGTM

@derkmed derkmed requested a review from Purg May 24, 2024 18:19
@Purg Purg merged commit 6bfc405 into PTG-Kitware:master May 24, 2024
3 checks passed
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