Policy Manager and NLG modules for MultiWOZ

[ekmb]

A rule-based policy manager and natural language generation modules added to complete the goal-oriented dialogue pipeline on MultiWOZ dataset.

The complete pipeline for Multiwoz looks like this:

User utterance => DST module (pre-trained TRADE) => rule-based PM => rule-based NLG => system reply
With the added example users can have a conversation with the system and analyze the results of each of the dialogue modules separately.

Signed-off-by: Evelina Bakhturina [email protected]

[lgtm-com]

This pull request introduces 23 alerts when merging a7b0ab5 into 78b6bef - view on LGTM.com

new alerts:

• 15 for Unused import
• 8 for Wrong name for an argument in a class instantiation

[lgtm-com]

This pull request introduces 23 alerts when merging 12256c9 into 78b6bef - view on LGTM.com

new alerts:

• 15 for Unused import
• 8 for Wrong name for an argument in a class instantiation

[lgtm-com]

This pull request introduces 1 alert when merging 7964c7e into 78b6bef - view on LGTM.com

new alerts:

• 1 for Wrong name for an argument in a class instantiation

[lgtm-com]

This pull request introduces 10 alerts when merging 307b942 into 1273d8f - view on LGTM.com

new alerts:

• 7 for Variable defined multiple times
• 2 for Unused import
• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 1 alert when merging b752fec into 1273d8f - view on LGTM.com

new alerts:

• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 1 alert when merging 800b771 into 9a1366c - view on LGTM.com

new alerts:

• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 1 alert when merging d913c65 into 1ae4aa9 - view on LGTM.com

new alerts:

• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 1 alert when merging 6f3980b into 1ae4aa9 - view on LGTM.com

new alerts:

• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 2 alerts when merging 0fc5372 into 1ae4aa9 - view on LGTM.com

new alerts:

• 1 for Unused import
• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 2 alerts when merging 7b9a899 into 1273aff - view on LGTM.com

new alerts:

• 1 for Unused import
• 1 for Non-iterable used in for loop

[lgtm-com]

This pull request introduces 1 alert when merging 9d97337 into e14590b - view on LGTM.com

new alerts:

• 1 for Unused import

[tkornuta-nvidia]

Decided to go with the current types

TODO (out-of-scope of this PR):

1. unit test for "neural tuple"
2. refactoring and unification of EncoderRNN