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

feat: use custom http_client #163

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Conversation

leseb
Copy link
Contributor

@leseb leseb commented Oct 29, 2024

252238b feat: use custom http_client

commit 252238b
Author: Sébastien Han seb@redhat.com
Date: Tue Oct 29 17:56:26 2024 +0100

feat: use custom http_client

This commit adds the ability to pass a custom HTTP client to the
MT-Bench evaluator. This is handy when using custom certificates when
interacting with the judge model serving endpoint.

Signed-off-by: Sébastien Han <seb@redhat.com>

@mergify mergify bot added documentation Improvements or additions to documentation testing Relates to testing ci-failure dependencies Pull requests that update a dependency file labels Oct 29, 2024
@leseb leseb force-pushed the http-client-endpoint branch from 2b8c46a to d11a178 Compare October 29, 2024 17:01
@mergify mergify bot added CI/CD Affects CI/CD configuration ci-failure and removed ci-failure labels Oct 29, 2024
@leseb leseb force-pushed the http-client-endpoint branch from d11a178 to 252238b Compare October 29, 2024 17:03
@mergify mergify bot removed the ci-failure label Oct 29, 2024
@nathan-weinberg nathan-weinberg requested a review from a team October 30, 2024 14:45
Copy link
Member

@nathan-weinberg nathan-weinberg left a comment

Choose a reason for hiding this comment

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

Code LGTM, passing CI, and we are already installing httpx-0.27.2 so no issues with a new dependency - thanks for the contribution @leseb!

@nathan-weinberg nathan-weinberg requested a review from a team October 30, 2024 14:51
@mergify mergify bot added the one-approval label Oct 30, 2024
@leseb leseb force-pushed the http-client-endpoint branch from 252238b to 26cc668 Compare October 31, 2024 08:39
This commit adds the ability to pass a custom HTTP client to the
MT-Bench evaluator. This is handy when using custom certificates when
interacting with the judge model serving endpoint.

Signed-off-by: Sébastien Han <seb@redhat.com>
@leseb leseb force-pushed the http-client-endpoint branch from 26cc668 to 010ff8c Compare October 31, 2024 08:40
@leseb leseb requested a review from danmcp October 31, 2024 09:38
@danmcp danmcp removed the request for review from a team October 31, 2024 11:18
@mergify mergify bot merged commit bd42ab8 into instructlab:main Oct 31, 2024
15 checks passed
@mergify mergify bot removed the one-approval label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Affects CI/CD configuration dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation testing Relates to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants