-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
FRv2.1-preview.2 update #10805
FRv2.1-preview.2 update #10805
Conversation
Swagger Validation Report
|
Rule | Message |
---|---|
The enum types should have x-ms-enum type extension set with appropriate options. Property name: state New: FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json#L1785 |
|
The enum types should have x-ms-enum type extension set with appropriate options. Property name: valueSelectionMark New: FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json#L1998 |
|
Guid used in model definition 'TrainResult' for property 'modelId'. Usage of Guid is not recommanded. If GUIDs are absolutely required in your service, please get sign off from the Azure API review board. New: FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json#L1174 |
|
Guid used in model definition 'DocumentResult' for property 'modelId'. Usage of Guid is not recommanded. If GUIDs are absolutely required in your service, please get sign off from the Azure API review board. New: FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json#L1913 |
️️✔️
Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️
ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️
SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️
[Staging] Cross Version BreakingChange (Base on preview version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️
[Staging] Cross Version BreakingChange (Base on stable version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️
CredScan succeeded [Detail] [Expand]
There is no credential detected.
Azure Pipelines successfully started running 1 pipeline(s). |
Azure CLI Extension Generation
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-java
|
azure-sdk-for-net
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-go
|
azure-resource-manager-schemas
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-python
|
azure-sdk-for-js
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-python-track2
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
Trenton Generation
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
…into FRv21UpdatePreview2
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
...ation/cognitiveservices/data-plane/FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json
Outdated
Show resolved
Hide resolved
...ation/cognitiveservices/data-plane/FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json
Outdated
Show resolved
Hide resolved
...ation/cognitiveservices/data-plane/FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json
Outdated
Show resolved
Hide resolved
...ation/cognitiveservices/data-plane/FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json
Outdated
Show resolved
Hide resolved
…into FRv21UpdatePreview2
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
…re-rest-api-specs into FRv21UpdatePreview2
Azure Pipelines successfully started running 1 pipeline(s). |
...ation/cognitiveservices/data-plane/FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json
Outdated
Show resolved
Hide resolved
...ation/cognitiveservices/data-plane/FormRecognizer/preview/v2.1-preview.2/FormRecognizer.json
Outdated
Show resolved
Hide resolved
…into FRv21UpdatePreview2
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
@ramparab do you have an ETA of when this PR is going to be merged? |
} | ||
} | ||
}, | ||
"Language": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems Japanese ("ja") should be included for this preview: https://docs.microsoft.com/en-us/azure/cognitive-services/form-recognizer/language-support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for catching this. @ramparab can you please fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ramparab I think this is the last thing to be resolved before this PR can be approved and merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@ramparab now that the service finished the deployment of 2.1-preview.2 and the SDKs are out too, do you have an ETA of when this PR is going to be ready and merged? |
Hi, @ramparab. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove |
…pecs into FRv21UpdatePreview2
…into FRv21UpdatePreview2
FRv2.1-preview.2 update