Skip to content
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

deprecate pynio backend #7301

Merged
merged 5 commits into from
Nov 26, 2022
Merged

deprecate pynio backend #7301

merged 5 commits into from
Nov 26, 2022

Conversation

jhamman
Copy link
Member

@jhamman jhamman commented Nov 19, 2022

This PR finally deprecates the PyNIO backend. PyNIO is technically in maintenance mode but it hasn't had any maintenance in 4+ years. Its conda packages cannot be installed in any of our test environments. I have added a future warning to the NioDataStore.__init__ method and noted the deprecation in the IO docs.

@github-actions github-actions bot added CI Continuous Integration tools dependencies Pull requests that update a dependency file io topic-backends labels Nov 19, 2022
xarray/backends/pynio_.py Outdated Show resolved Hide resolved
@headtr1ck headtr1ck added the plan to merge Final call for comments label Nov 19, 2022
@headtr1ck
Copy link
Collaborator

Maybe we should remove the pynio backend tests alltogether since they will be skipped anyway if no env has pynio enabled anymore?

@jhamman
Copy link
Member Author

jhamman commented Nov 19, 2022

Maybe we should remove the pynio backend tests all together since they will be skipped anyway if no env has pynio enabled anymore?

I think we should keep them in the test suite as long as the backend is in the code. Although we able to install pynio in our test envs, I suspect it is still possible to construct an environment centered around the pynio that would still work.

@Illviljan Illviljan merged commit fb42ef9 into pydata:main Nov 26, 2022
@Illviljan
Copy link
Contributor

Thanks, @jhamman !

dcherian added a commit to dcherian/xarray that referenced this pull request Dec 2, 2022
* upstream/main: (39 commits)
  Support the new compression argument in netCDF4 > 1.6.0 (pydata#6981)
  Remove setuptools-scm-git-archive, require setuptools-scm>=7 (pydata#7253)
  Fix mypy failures (pydata#7343)
  Docs: add example of writing and reading groups to netcdf (pydata#7338)
  Reset file pointer to 0 when reading file stream (pydata#7304)
  Enable mypy warn unused ignores (pydata#7335)
  Optimize some copying (pydata#7209)
  Add parse_dims func (pydata#7051)
  Fix coordinate attr handling in `xr.where(..., keep_attrs=True)` (pydata#7229)
  Remove code used to support h5py<2.10.0 (pydata#7334)
  [pre-commit.ci] pre-commit autoupdate (pydata#7330)
  Fix PR number in what’s new (pydata#7331)
  Enable `origin` and `offset` arguments in `resample` (pydata#7284)
  fix doctests: supress urllib3 warning (pydata#7326)
  fix flake8 config (pydata#7321)
  implement Zarr v3 spec support (pydata#6475)
  Fix polyval overloads (pydata#7315)
  deprecate pynio backend (pydata#7301)
  mypy - Remove some ignored packages and modules (pydata#7319)
  Switch to T_DataArray in .coords (pydata#7285)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration tools dependencies Pull requests that update a dependency file io plan to merge Final call for comments topic-backends
Projects
None yet
Development

Successfully merging this pull request may close these issues.

deprecate pynio backend
3 participants