-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test minimum and maximum supported dependencies #918
Changes from all commits
f3a90f9
33bb1dc
81082c3
b4dc27b
16e75d4
d8f3d3b
398a4fe
73c889c
04c2f5f
59e5a37
46da56a
c03986a
ff70198
bd3fd52
b1f814a
d863a49
4c8e94c
df1d528
b8d0241
a63746f
4fec786
c3a67d8
0827645
1267aab
d835a41
e471416
571f5c0
b3f047c
82ab059
ca431af
57035bd
a8fba18
b5c7437
c06e19c
821c4ab
e517d34
57e55aa
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,30 +17,30 @@ classifiers = [ | |
'Programming Language :: Python :: Implementation :: PyPy', | ||
] | ||
dependencies = [ | ||
'efficientnet-pytorch==0.7.1', | ||
'huggingface-hub>=0.24.6', | ||
'numpy', | ||
'pillow', | ||
'pretrainedmodels==0.7.4', | ||
'six', | ||
'timm==0.9.7', | ||
'torch', | ||
'torchvision>=0.5', | ||
'tqdm', | ||
'efficientnet-pytorch>=0.6.1', | ||
'huggingface-hub>=0.24', | ||
'numpy>=1.19.3', | ||
'pillow>=8', | ||
'pretrainedmodels>=0.7.1', | ||
'six>=1.5', | ||
'timm>=0.9', | ||
adamjstewart marked this conversation as resolved.
Show resolved
Hide resolved
|
||
'torch>=1.8', | ||
'torchvision>=0.9', | ||
'tqdm>=4.42.1', | ||
] | ||
dynamic = ['version'] | ||
|
||
[project.optional-dependencies] | ||
docs = [ | ||
'autodocsumm', | ||
'huggingface-hub', | ||
'six==1.15.0', | ||
'sphinx<7', | ||
'sphinx-book-theme==1.1.2', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should probably test that newer versions of sphinx and sphinx-book-theme actually work, but this requires #920. |
||
'six', | ||
'sphinx', | ||
'sphinx-book-theme', | ||
] | ||
test = [ | ||
'pytest', | ||
'ruff==0.5.2', | ||
'ruff', | ||
] | ||
|
||
[project.urls] | ||
|
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
autodocsumm==0.2.13 | ||
huggingface-hub==0.24.6 | ||
six==1.16.0 | ||
sphinx==8.0.2 | ||
sphinx-book-theme==1.1.3 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
efficientnet-pytorch==0.6.1 | ||
huggingface-hub==0.24.0 | ||
numpy==1.19.3 | ||
pillow==8.0.0 | ||
pretrainedmodels==0.7.1 | ||
six==1.5.0 | ||
timm==0.9.0 | ||
torch==1.8.0 | ||
torchvision==0.9.0 | ||
tqdm==4.42.1 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
efficientnet-pytorch==0.7.1 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why pin the required? Shouldn't keep it and install the latest version automatically? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This file is only used for CI and is controlled by dependabot. The dependencies used during installation are found in pyproject.toml. |
||
huggingface_hub==0.24.6 | ||
numpy==2.1.1 | ||
pillow==10.4.0 | ||
pretrainedmodels==0.7.4 | ||
six==1.16.0 | ||
timm==1.0.9 | ||
torch==2.4.1 | ||
torchvision==0.19.1 | ||
tqdm==4.66.5 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
pytest==8.3.3 | ||
ruff==0.6.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of hard running it here, what do you think about setting up the https://pre-commit.com/ here, then after you guys set up the https://pre-commit.ci into the repo, so it enables the auto-fix feature on PR's...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have mixed feelings about this. I like the ability to auto-fix issues, just not auto-commits. I'm on the fence. We haven't enabled this in TorchGeo either.