Skip to content

Commit eadd4cd

Browse files
committed
Remove [test] package from sklearn instrumentation
Fixes #2218
1 parent 1e0b11f commit eadd4cd

File tree

3 files changed

+24
-6
lines changed

3 files changed

+24
-6
lines changed

instrumentation/opentelemetry-instrumentation-sklearn/pyproject.toml

-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,6 @@ dependencies = [
3232
instruments = [
3333
"scikit-learn ~= 0.24.0",
3434
]
35-
test = [
36-
"opentelemetry-instrumentation-sklearn[instruments]",
37-
"opentelemetry-test-utils == 0.45b0.dev",
38-
]
3935

4036
[project.entry-points.opentelemetry_instrumentor]
4137
sklearn = "opentelemetry.instrumentation.sklearn:SklearnInstrumentor"
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
asgiref==3.7.2
2+
attrs==23.2.0
3+
Deprecated==1.2.14
4+
importlib-metadata==6.11.0
5+
iniconfig==2.0.0
6+
joblib==1.3.2
7+
numpy==1.24.4
8+
packaging==23.2
9+
pluggy==1.4.0
10+
py==1.11.0
11+
py-cpuinfo==9.0.0
12+
pytest==7.1.3
13+
pytest-benchmark==4.0.0
14+
scikit-learn==0.24.2
15+
scipy==1.10.1
16+
threadpoolctl==3.3.0
17+
tomli==2.0.1
18+
typing_extensions==4.10.0
19+
wrapt==1.16.0
20+
zipp==3.17.0
21+
-e opentelemetry-instrumentation
22+
-e instrumentation/opentelemetry-instrumentation-sklearn

tox.ini

+2-2
Original file line numberDiff line numberDiff line change
@@ -412,7 +412,7 @@ commands_pre =
412412

413413
prometheus: pip install {toxinidir}/exporter/opentelemetry-exporter-prometheus-remote-write[test]
414414

415-
sklearn: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-sklearn[test]
415+
sklearn: pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-sklearn/test-requirements.txt
416416

417417
sqlalchemy-{11,14}: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-sqlalchemy[test]
418418

@@ -550,7 +550,7 @@ commands_pre =
550550
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-celery[test]
551551
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-pika[test]
552552
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-aio-pika[test]
553-
; python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-sklearn[test]
553+
; pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-sklearn/test-requirements.txt
554554
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-redis[test]
555555
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-remoulade[test]
556556
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-fastapi[test]

0 commit comments

Comments
 (0)