Skip to content

Commit 4c63558

Browse files
committed
fix: dependence bugs and cleanup
* pandas[excel]<2 doesn't work, go back to xlrd and comment * add mock to changehc * add requests
1 parent 45cfd8f commit 4c63558

File tree

4 files changed

+14
-21
lines changed

4 files changed

+14
-21
lines changed

_delphi_utils_python/pyproject.toml

+7-9
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,12 @@ version = "0.3.24"
88
description = "Shared Utility Functions for Indicators"
99
readme = "README.md"
1010
requires-python = "== 3.8.*"
11-
license = {text = "MIT License"}
11+
license = { text = "MIT License" }
1212
classifiers = [
1313
"Development Status :: 5 - Production/Stable",
1414
"Intended Audience :: Developers",
1515
"Programming Language :: Python :: 3.8",
16-
"License :: MIT"
16+
"License :: MIT",
1717
]
1818
dependencies = [
1919
"boto3",
@@ -23,16 +23,17 @@ dependencies = [
2323
"gitpython",
2424
"importlib_resources>=1.3",
2525
"numpy",
26-
"pandas[excel]>=1.1.0",
26+
"pandas>=1.1.0",
27+
"requests",
2728
"slackclient",
28-
"scs<3.2.6", # TODO: remove this ; it is a cvxpy dependency, and the excluded version appears to break our jenkins build. see: https://github.com/cvxgrp/scs/issues/283
29+
"scs<3.2.6", # TODO: remove this ; it is a cvxpy dependency, and the excluded version appears to break our jenkins build. see: https://github.com/cvxgrp/scs/issues/283
2930
"structlog",
31+
"xlrd", # needed by Pandas to read Excel files
3032
]
3133

3234
[project.urls]
3335
Homepage = "https://github.com/cmu-delphi/covidcast-indicators"
3436

35-
3637
[project.optional-dependencies]
3738
dev = [
3839
"darker[isort]~=2.1.1",
@@ -45,7 +46,7 @@ dev = [
4546
"requests-mock",
4647
"freezegun",
4748
]
48-
49+
flash = ["scipy"]
4950

5051
[tool.setuptools.packages.find]
5152
where = ["."]
@@ -54,6 +55,3 @@ namespaces = true
5455

5556
[tool.setuptools.package-data]
5657
"delphi_utils.data" = ["20*/*.csv"]
57-
58-
59-

_template_python/pyproject.toml

+5-11
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,21 @@ build-backend = "setuptools.build_meta"
44

55
[project]
66
name = "delphi_NAME"
7-
version = "0.3.24"
8-
description = "HORT DESCRIPTION"
7+
version = "0.1.0"
8+
description = "SHORT DESCRIPTION"
99
readme = "README.md"
1010
requires-python = "== 3.8.*"
11-
license = {text = "MIT License"}
11+
license = { text = "MIT License" }
1212
classifiers = [
1313
"Development Status :: 5 - Production/Stable",
1414
"Intended Audience :: Developers",
1515
"Programming Language :: Python :: 3.8",
16-
"License :: MIT"
16+
"License :: MIT",
1717
]
18-
1918
dependencies = [
20-
"covidcast",
2119
"delphi-utils",
2220
"numpy",
23-
"pandas[excel]>=1.1.0",
21+
"pandas>=1.1.0",
2422
]
2523

2624
[project.urls]
@@ -39,10 +37,6 @@ dev = [
3937
"freezegun",
4038
]
4139

42-
4340
[tool.setuptools.packages.find]
4441
where = ["."]
4542
namespaces = true
46-
47-
48-

changehc/setup.py

+1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
"covidcast",
77
"darker[isort]~=2.1.1",
88
"delphi-utils",
9+
"mock",
910
"moto~=4.2.14",
1011
"numpy",
1112
"pandas",

quidel_covidtest/setup.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
"pylint==2.8.3",
1616
"pytest-cov",
1717
"pytest",
18-
"xlrd==1.2.0",
18+
"xlrd==1.2.0", # needed by Pandas to read Excel files
1919
]
2020

2121
setup(

0 commit comments

Comments
 (0)