Skip to content

Commit c78906d

Browse files
author
Andrei Bratu
committed
PR feedback
1 parent 099ae1d commit c78906d

File tree

4 files changed

+226
-270
lines changed

4 files changed

+226
-270
lines changed

poetry.lock

Lines changed: 13 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ Repository = 'https://github.com/humanloop/humanloop-python'
3535

3636
[tool.poetry.dependencies]
3737
python = ">=3.9,<4"
38-
deepdiff = "^6.7.1"
38+
deepdiff = "^8.2.0"
3939
httpx = ">=0.21.2"
4040
httpx-sse = "0.4.0"
4141
mmh3 = "^5.1.0"

src/humanloop/eval_utils/run.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
import time
2020
import types
2121
import typing
22-
from concurrent.futures import ThreadPoolExecutor, as_completed
22+
from concurrent.futures import ThreadPoolExecutor
2323
from datetime import datetime
2424
from functools import partial
2525
from logging import INFO
@@ -367,15 +367,8 @@ def upload_callback(log_id: str):
367367
)
368368

369369
with ThreadPoolExecutor(max_workers=workers) as executor:
370-
futures = []
371370
for datapoint in hl_dataset.datapoints:
372-
futures.append(executor.submit(_process_datapoint, datapoint))
373-
# Program hangs if any uncaught exceptions are not handled here
374-
for future in as_completed(futures):
375-
try:
376-
future.result()
377-
except Exception:
378-
pass
371+
executor.submit(_process_datapoint, datapoint)
379372

380373
stats = _wait_for_evaluation_to_complete(
381374
client=client,

0 commit comments

Comments
 (0)