[aIbrahiim]

Fixes: #30799

PostCommit Python Dependency failing on embeddings + tft-121.

HF: with a token set the provider can still reject, and it was treated {"error":...} as embeddings ((5, False)). Raise on that, and skip the live test if the API isn't usable.

TFT: vocab stdout tests flake on print order so switch those two to assertCountEqual.

[github-actions]

📋 Review Summary

This PR improves test stability by making stdout assertions in mltransform_test.py order-independent and adding proper HTTP status checking, error payload validation, and pre-flight availability checks for the Hugging Face Inference API in huggingface.py and huggingface_test.py. The changes are clean, well-scoped, and effectively address potential test flakes.

🔍 General Feedback

• In mltransform_test.py, switching to assertCountEqual for vocabulary computation tests makes sense since pipeline execution does not guarantee element print ordering (consider applying the same to test_mltransform_scale_to_0_1 if it can also experience out-of-order execution).
• The error handling additions in _InferenceAPIHandler.run_inference (raise_for_status() and checking for 'error' in the response dict) prevent API error payloads from silently passing through as embedding outputs.

---

💡 Tip: Comment /gemini review on this pull request to trigger a new Gemini code review.

[github-actions]

Assigning reviewers:

R: @jrmccluskey for label python.
Backup reviewer: tvalentyn (if the main reviewer is unresponsive, you can escalate to them by tagging their username)

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

• stop reviewer notifications - opt out of the automated review tooling
• remind me after tests pass - tag the comment author after tests pass
• waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

[aIbrahiim]

R: @Abacn

[aIbrahiim]

assign to next reviewer

[github-actions]

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment assign set of reviewers

[codecov]

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.09%. Comparing base (12bbb17) to head (895e93e).
⚠️ Report is 46 commits behind head on master.

Files with missing lines	Patch %	Lines
...pache_beam/ml/transforms/embeddings/huggingface.py	0.00%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #40953      +/-   ##
==========================================
+ Coverage   79.83%   80.09%   +0.26%     
==========================================
  Files         494      494              
  Lines       83575    83614      +39     
==========================================
+ Hits        66722    66972     +250     
+ Misses      16853    16642     -211     

Flag	Coverage Δ
python	80.09% <0.00%> (+0.26%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[aIbrahiim]

assign set of reviewers

[github-actions]

Assigning reviewers:

R: @jrmccluskey for label python.
Backup reviewer: tvalentyn (if the main reviewer is unresponsive, you can escalate to them by tagging their username)

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

• stop reviewer notifications - opt out of the automated review tooling
• remind me after tests pass - tag the comment author after tests pass
• waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

[aIbrahiim]

PostCommit Python Dependency checks passed