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

Enable Conversations live tests #26346

Merged
merged 4 commits into from
Jan 14, 2022
Merged

Enable Conversations live tests #26346

merged 4 commits into from
Jan 14, 2022

Conversation

heaths
Copy link
Member

@heaths heaths commented Jan 14, 2022

The pipeline variable group has already been updated.

@heaths
Copy link
Member Author

heaths commented Jan 14, 2022

/azp run net - cognitivelanguage - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@heaths
Copy link
Member Author

heaths commented Jan 14, 2022

/azp run net - cognitiveservices - tests

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@heaths
Copy link
Member Author

heaths commented Jan 14, 2022

/azp run net - cognitivelanguage - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@heaths
Copy link
Member Author

heaths commented Jan 14, 2022

Good news: this almost works! @AhmedLeithy the test AnalyzeConversationWithLanguage is failing with 400 BadRequest but the message isn't very specific as to why. Could you look into this, please? If we can get that fixed, these changes should re-enable the live tests.

/cc @annatisch @ZulaMostafa @mshaban-msft

Disable a few tests until Azure#26379 is resolved so we can get nightly live tests working again.
@heaths
Copy link
Member Author

heaths commented Jan 14, 2022

/azp run net - cognitivelanguage - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Had a second result come back that was 0.30 confidence, which I've seen in the past as well. Tends to vary whenever the model is retrained. Ask for a higher threshold now.
@heaths
Copy link
Member Author

heaths commented Jan 14, 2022

/azp run net - cognitivelanguage - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@heaths heaths merged commit ff8e1b2 into Azure:main Jan 14, 2022
@heaths heaths deleted the clu-tests branch January 14, 2022 23:47
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.

6 participants