Skip to content

Commit

Permalink
fix: ensure rest api urls include custom root path (#4137)
Browse files Browse the repository at this point in the history
  • Loading branch information
axiomofjoy authored Aug 6, 2024
1 parent c18636e commit 9550a7e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/phoenix/session/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -324,7 +324,7 @@ def _get_dataset_id_by_name(self, name: str) -> str:
Dataset: The dataset object.
"""
response = self._client.get(
urljoin(self._base_url, "/v1/datasets"),
urljoin(self._base_url, "v1/datasets"),
params={"name": name},
)
response.raise_for_status()
Expand Down Expand Up @@ -366,7 +366,7 @@ def get_dataset(
raise ValueError("Dataset id or name must be provided.")

response = self._client.get(
urljoin(self._base_url, f"/v1/datasets/{quote(id)}/examples"),
urljoin(self._base_url, f"v1/datasets/{quote(id)}/examples"),
params={"version_id": version_id} if version_id else None,
)
response.raise_for_status()
Expand Down

0 comments on commit 9550a7e

Please sign in to comment.