-
-
Notifications
You must be signed in to change notification settings - Fork 54
Updates from package template #889
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
Merged
Merged
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
b07b15b
Update cruft with batchpr
Cadair 3ca3829
Resolve conflicts
Cadair 0f3d6a7
bump checkout
Cadair f3e526d
Update pyproject.toml
nabobalis 8139b7b
Update minimum version requirements in changelog
nabobalis fba04e5
Update gwcs dependency version
nabobalis 6924883
Update 889.breaking.rst
nabobalis ffba363
Update 889.breaking.rst
nabobalis 4ae23e1
Update gwcs dependency version to 0.20.0
nabobalis e3438f9
Update pyproject.toml
nabobalis 4d7a04e
Cleanup oldestdeps
Cadair 5df8171
Apply suggestion from @Cadair
Cadair 672704f
Apply suggestion from @Cadair
Cadair 210728a
Apply suggestion from @Cadair
Cadair 457084b
6.0 I guess
Cadair 9e04c6a
SunPy
Cadair 7ee8d75
Roll back to 3.11 for a bit
Cadair 397b904
Update .cruft.json
Cadair daf30b5
Update installation.rst
Cadair 059ac9c
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| Increased minimum version of Python to 3.11. | ||
| Increased minimum version of NumPy to 1.26.0. | ||
| Increased minimum version of SciPy to 1.12.0. | ||
| Increased minimum version of astropy to 6.0.0. | ||
| Increased minimum version of gwcs to 0.20.0. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,52 +9,54 @@ build-backend = "setuptools.build_meta" | |
| [project] | ||
| name = "ndcube" | ||
| description = "A package for multi-dimensional contiguous and non-contiguous coordinate aware arrays." | ||
| requires-python = ">=3.10" | ||
| requires-python = ">=3.11" | ||
| readme = { file = "README.rst", content-type = "text/x-rst" } | ||
| license = { file = "licenses/LICENSE.rst" } | ||
| authors = [ | ||
| { name = "The SunPy Community", email = "[email protected]" }, | ||
| ] | ||
| dependencies = [ | ||
| "astropy>=5.0.6,!=5.1.0", | ||
| "gwcs>=0.18", | ||
| "numpy>=1.23.0", | ||
| "scipy>=1.8.0", | ||
| "astropy>=6.0.0", | ||
| "gwcs>=0.20.0", | ||
| "numpy>=1.26.0", | ||
| "scipy>=1.12.0", | ||
| ] | ||
| dynamic = ["version"] | ||
|
|
||
| [project.optional-dependencies] | ||
| # These are truly optional deps or deps only used for testing where we | ||
| # want oldestdeps to test them | ||
| tests-optional = [ | ||
| "dask>=2023.10.0", | ||
| "specutils>=1.12.0", | ||
| "sunpy>=6.1.0", | ||
| ] | ||
| tests = [ | ||
| "dask", | ||
| "ndcube[all,tests-optional]", | ||
| "pytest-astropy", | ||
| "pytest-cov", | ||
| "pytest-doctestplus", | ||
| "pytest-mpl>=0.12", | ||
| "pytest-xdist", | ||
| "pytest", | ||
| "pytest-memray; sys_platform != 'win32'", | ||
| "scipy", | ||
| "specutils", | ||
| "sunpy>=5.0.0", | ||
| ] | ||
| docs = [ | ||
| "ndcube[all,tests-optional]", | ||
| "sphinx", | ||
| "sphinx-automodapi", | ||
| "sunpy-sphinx-theme", | ||
| "packaging", | ||
| "matplotlib", | ||
| "mpl-animators>=1.0", | ||
| "sphinx-changelog>=1.1.0", | ||
| "sphinx-gallery", | ||
| "sphinxext-opengraph", | ||
| "sunpy>=5.0.0", | ||
| ] | ||
| plotting = [ | ||
| "matplotlib>=3.5.0", | ||
| "matplotlib>=3.8.0", | ||
| "mpl_animators>=1.0", | ||
| ] | ||
| reproject = [ | ||
| "reproject>=0.7.1", | ||
| "reproject>=0.13", # first version with 3.12 support | ||
nabobalis marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| ] | ||
| all = [ | ||
| "ndcube[plotting,reproject]", | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,9 +3,9 @@ min_version = 4.0 | |
| requires = | ||
| tox-pypi-filter>=0.14 | ||
| envlist = | ||
| py{310,311,312} | ||
| py312-devdeps | ||
| py310-oldestdeps | ||
| py{311,312,313,314} | ||
| py314-devdeps | ||
| py311-oldestdeps | ||
| codestyle | ||
| build_docs | ||
|
|
||
|
|
@@ -66,7 +66,7 @@ extras = | |
| reproject | ||
| tests | ||
| commands_pre = | ||
| oldestdeps: minimum_dependencies ndcube --filename requirements-min.txt | ||
| oldestdeps: minimum_dependencies ndcube --extras plotting reproject tests-optional --filename requirements-min.txt | ||
|
Member
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 make sure we do this for packages in the future. |
||
| oldestdeps: pip install -r requirements-min.txt | ||
| oldestdeps: python -c "import astropy.time; astropy.time.update_leap_seconds()" | ||
| pip freeze --all --no-input | ||
|
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.