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

This fixes an error that was caused by the breaking API change from t… #94

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

Conversation

kyichii
Copy link

@kyichii kyichii commented Dec 16, 2020

…he python SDK commit: Azure/azure-sdk-for-python@99668db

Purpose

The following changes resolve a regression in the samples that occurred during the in the following commit with API breaking changes. Azure/azure-sdk-for-python@99668db

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
  • in samples/vision/computer_vision_extract_text.py
    • change 'endpoint' to valid computer-vision resource endpoint that you have access to
    • change 'key' to valid subscription key for the above computer-vision resource
  • change to director samples/vision/
  • Run the code with 'python computer_vision_extract_text.py'

What to Check

Verify that the following are valid

  • No Module Errors, a valid computer vision result is received.

Other Information

This is related to a Service Request opened by a customer indicating that they received errors when attempting to run the computer_vision_extract_text.py sample. This PR should resolve the issue from the API breaking changes that previously occurred.

@ghost
Copy link

ghost commented Dec 16, 2020

CLA assistant check
All CLA requirements met.

…demo. The previous URL does not appear to be avaialble

This fixes an error that was caused by the breaking API change from the python SDK commit:  Azure/azure-sdk-for-python@99668db
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