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

[formrecognizer] Update README to include FR preview.3 information #17649

Merged
merged 1 commit into from
Mar 30, 2021

Conversation

catalinaperalta
Copy link
Member

Fixes #17142

@catalinaperalta catalinaperalta marked this pull request as ready for review March 30, 2021 17:04
for id_document in result:
for name, field in id_document.fields.items():
print("{}: {} has confidence {}".format(name, field.value, field.confidence))
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we've talked about refactoring the examples section once we had several prebuilts since these samples all pretty much look the same and make the readme pretty lengthy. not blocking on this PR, but maybe open an issue so we can discuss how we want to do it across langs?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed! I didnt want to skip id docs just now since all the other models still have their examples. Here's the issue: #17666

@catalinaperalta catalinaperalta merged commit d678b06 into Azure:master Mar 30, 2021
@catalinaperalta catalinaperalta deleted the updateReadmePreview3 branch March 30, 2021 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[formrecognizer] update readme with preview.3 features
2 participants