Skip to content

fix: let Document Intelligence use SDK default API version#1906

Open
he-yufeng wants to merge 1 commit into
microsoft:mainfrom
he-yufeng:fix/docintel-default-api-version
Open

fix: let Document Intelligence use SDK default API version#1906
he-yufeng wants to merge 1 commit into
microsoft:mainfrom
he-yufeng:fix/docintel-default-api-version

Conversation

@he-yufeng
Copy link
Copy Markdown

Fixes #1904.

Summary

  • stop forcing the older Document Intelligence preview API version by default
  • only pass api_version to DocumentIntelligenceClient when the caller supplies one
  • add focused constructor tests that do not call Azure

Verified

  • .venv\Scripts\python.exe -m pytest packages\markitdown\tests\test_docintel_html.py -q
  • .venv\Scripts\python.exe -m py_compile packages\markitdown\src\markitdown\converters_doc_intel_converter.py packages\markitdown\tests\test_docintel_html.py
  • git diff --check

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.

DocumentIntelligenceConverter should not set a default api_version

1 participant