Skip to content

Commit

Permalink
TST: Fix devdeps job (#2572)
Browse files Browse the repository at this point in the history
* TST: Add pyerfa-dev to devdeps
and remove unnecessary tox requirements.

BLD: No longer need to specify wheel in build requirements.

* Temporarily disable numpy-dev in devdeps

* BUG: Fix catalog test
  • Loading branch information
pllim authored Jan 9, 2024
1 parent 410d2f2 commit d91877a
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 12 deletions.
2 changes: 1 addition & 1 deletion jdaviz/configs/imviz/tests/test_catalogs.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ def test_plugin_image_with_result(self, imviz_helper, tmp_path):
self.imviz = imviz_helper

catalogs_plugin = self.imviz.app.get_tray_item_from_name('imviz-catalogs')
catalogs_plugin._obj.plugin_opened = True
catalogs_plugin.plugin_opened = True
# This basically calls the following under the hood:
# skycoord_center = SkyCoord(6.62754354, 1.54466139, unit="deg")
# zoom_radius = r_max = 3 * u.arcmin
Expand Down
3 changes: 1 addition & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,7 @@ roman = [
[build-system]
requires = [
"setuptools>=61.2",
"setuptools_scm",
"wheel",
"setuptools_scm"
]
build-backend = "setuptools.build_meta"

Expand Down
13 changes: 4 additions & 9 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,14 @@ envlist =
codestyle
pep517
securityaudit
requires =
setuptools >= 30.3.0
pip >= 19.3.1
isolated_build = true

[testenv]
# Suppress display of matplotlib plots generated during docs build
setenv =
MPLBACKEND=agg
JUPYTER_PLATFORM_DIRS=1
devdeps: PIP_EXTRA_INDEX_URL = https://pypi.anaconda.org/astropy/simple https://pypi.anaconda.org/scientific-python-nightly-wheels/simple
devdeps: PIP_EXTRA_INDEX_URL = https://pypi.anaconda.org/astropy/simple https://pypi.anaconda.org/liberfa/simple https://pypi.anaconda.org/scientific-python-nightly-wheels/simple

# Pass through the following environment variables which may be needed for the CI
passenv = HOME,WINDIR,LC_ALL,LC_CTYPE,CC,CI
Expand All @@ -42,11 +39,12 @@ description =
# The following provides some specific pinnings for key packages
deps =
# NOTE: Add/remove as needed
devdeps: numpy>=0.0.dev0
#devdeps: numpy>=0.0.dev0
devdeps: scipy>=0.0.dev0
devdeps: matplotlib>=0.0.dev0
devdeps: pandas>=0.0.dev0
devdeps: scikit-image>=0.0.dev0
devdeps: pyerfa>=0.0.dev0
devdeps: astropy>=0.0.dev0
devdeps: git+https://github.com/astropy/regions.git
devdeps: git+https://github.com/astropy/specutils.git
Expand All @@ -65,9 +63,6 @@ deps =
devdeps: git+https://github.com/glue-viz/glue-astronomy.git
# TODO: Enable this when specreduce becomes stable.
#devdeps: git+https://github.com/astropy/specreduce.git
# TODO: Remove both when we can, see https://github.com/glue-viz/glue-astronomy/issues/62
devdeps: git+https://github.com/radio-astro-tools/spectral-cube.git
devdeps: git+https://github.com/radio-astro-tools/radio-beam.git

# The following indicates which extras_require from pyproject.toml will be installed
extras =
Expand All @@ -77,7 +72,7 @@ extras =

commands =
# Force numpy-dev after matplotlib downgrades it (https://github.com/matplotlib/matplotlib/issues/26847)
devdeps: python -m pip install --pre --upgrade --extra-index-url https://pypi.anaconda.org/scientific-python-nightly-wheels/simple numpy
#devdeps: python -m pip install --pre --upgrade --extra-index-url https://pypi.anaconda.org/scientific-python-nightly-wheels/simple numpy
jupyter --paths
pip freeze
!cov: pytest --pyargs jdaviz {toxinidir}/docs {posargs}
Expand Down

0 comments on commit d91877a

Please sign in to comment.