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

fix Luis:build lost error message for some cases #1214

Merged
merged 2 commits into from
Apr 26, 2021

Conversation

cosmicshuai
Copy link
Contributor

Suppose we have a lu file to build

# Why
- help
- what can I say
- why do you need my name?
- why age?
- what do you need my profile for?
- why do you ask?
- why do you need that information? 

@ phraselist Test1
    - t1
    - q1

@ phraselist Test2
    - t2
    - q2

@ phraselist Test3
    - t3
    - q3

@ phraselist Test4
    - t4
    - q4

@ phraselist Test5
    - t5
    - q5

@ phraselist Test6
    - t6
    - q6

@ phraselist Test7
    - t7
    - q7

@ phraselist Test8
    - t8
    - q8

@ phraselist Test9
    - t9
    - q9

@ phraselist Test10
    - t10
    - q10

@ phraselist Test11
    - t11
    - q11

@ phraselist Test12
    - t12
    - q1

For previous bf-luis-cli build, this will throw an error:
Luis build failed: undefined

In this PR, the luis: build command will return a more detailed error message:
Luis build failed: [ERROR] intent Why has 12 phraselist descriptors (feature). At most 10 is allowed.

@munozemilio munozemilio merged commit c262712 into main Apr 26, 2021
@munozemilio munozemilio deleted the shuwan/fixLuisBuildError branch April 26, 2021 18:16
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.

None yet

2 participants