Skip to content

Conversation

mdrxy
Copy link
Collaborator

@mdrxy mdrxy commented Oct 16, 2025

plus tests minor rfc

@mdrxy mdrxy requested a review from ccurme as a code owner October 16, 2025 23:39
@github-actions github-actions bot added integration Related to a provider partner package integration fix labels Oct 16, 2025
Copy link

codspeed-hq bot commented Oct 16, 2025

CodSpeed Performance Report

Merging #33543 will not alter performance

Comparing mdrxy/xai-v1-tests (ec6a40e) with master (6997867)1

Summary

✅ 1 untouched
⏩ 33 skipped2

Footnotes

  1. No successful run was found on master (8fd54f1) during the generation of this report, so 6997867 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

  2. 33 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@mdrxy mdrxy changed the title fix(xai): include model_provider in reasoning_content fix(xai): inject model_provider in response_metadata Oct 16, 2025
@github-actions github-actions bot added fix and removed fix labels Oct 16, 2025
MODEL_NAME = "grok-4-fast-reasoning"


@pytest.mark.parametrize("output_version", ["", "v1"])
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
@pytest.mark.parametrize("output_version", ["", "v1"])
@pytest.mark.parametrize("output_version", ["v0", "v1"])

None also works for v0

@mdrxy mdrxy merged commit 8efa75d into master Oct 17, 2025
23 checks passed
@mdrxy mdrxy deleted the mdrxy/xai-v1-tests branch October 17, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix integration Related to a provider partner package integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants