Skip to content

Commit 847e8fa

Browse files
authored
Remove [test] package from sklearn instrumentation (#2303)
Fixes #2218
1 parent e7df01a commit 847e8fa

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
@@ -434,7 +434,7 @@ commands_pre =
434434

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

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

439439
sqlalchemy-0: pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-sqlalchemy/test-requirements-0.txt
440440
sqlalchemy-1: pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-sqlalchemy/test-requirements-1.txt
@@ -580,12 +580,12 @@ commands_pre =
580580
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-boto/test-requirements.txt
581581
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-pika[test]
582582
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-aio-pika/test-requirements-2.txt
583-
; python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-sklearn[test]
584583
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-redis/test-requirements.txt
585584
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-fastapi/test-requirements.txt
586585
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-jinja2/test-requirements.txt
587586
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-kafka-python/test-requirements.txt
588587
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-remoulade/test-requirements.txt
588+
; pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-sklearn/test-requirements.txt
589589
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-confluent-kafka[test]
590590
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-pymemcache/test-requirements-4.txt
591591
pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-logging/test-requirements.txt

0 commit comments

Comments
 (0)