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 cognitiveModels outdated json import files #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

milannakum
Copy link

Purpose

  • It saves users for users trying to use this samples to import project in Azure Language Studio. Currently the data format this sample uses is outdated and fails to successfully import.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe: Fixed the JSON file which fails to import in Azure Language Studio due to outdated format version.

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code
Go to Azure Language Studio and try to import the sample model JSON file to create new project, this will throw an error due to the outdated format used in current JSON file.

What to Check

Verify that the following are valid

  • Able to successfully import the project in Azure Language Studio from the JSON file cognitiveModels/FlightBooking.json.

Other Information

Refer updated documentation here for related changes: Data formats accepted by conversational language understanding

@milannakum
Copy link
Author

@microsoft-github-policy-service agree

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.

1 participant