FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[Bot-AI-QnA] Fix disparities by Batta32 · Pull Request #1214 · microsoft/botbuilder-java · GitHub

This repository was archived by the owner on Dec 4, 2023. It is now read-only.
/ botbuilder-java Public archive

[Bot-AI-QnA] Fix disparities - #1214

Merged
tracyboehrer (tracyboehrer) merged 6 commits into
microsoft:mainfrom
southworks:external/fix/southworks/libraries/bot-ai-qna/fix-disparities
May 27, 2021
Merged

[Bot-AI-QnA] Fix disparities#1214
tracyboehrer (tracyboehrer) merged 6 commits into
microsoft:mainfrom
southworks:external/fix/southworks/libraries/bot-ai-qna/fix-disparities

Conversation

Copy link
Copy Markdown
Contributor

Fix #1213
Related to #1166

Description

We compared the structure/code migration/unit tests/documentation of the bot-ai-qna library between Java and C# and we found disparities and fixes that this PR includes.

Specific Changes

  • Rename testData to testdata following other Java libraries
  • Fix javadocs/typos
  • Use of StringUtils instead of String to keep consistency
  • Fix scoreThreshold of an unit tests (0.4 -> 0.42)

Testing

mvn clean install passing successfully

tracyboehrer (tracyboehrer) merged commit c170250 into microsoft:main May 27, 2021
Martin Battaglino (Batta32) deleted the external/fix/southworks/libraries/bot-ai-qna/fix-disparities branch May 27, 2021 20:07
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SDK][Bot-AI-QnA] Disparities between C#/JavaScript and Java

2 participants


Back | FazBrowse Home | New Git URL