[pull] main from pydata:main #565
Merged
Conversation
This file contains 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
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
* Cleanup test_coding_times * Update test_coding_times.py
* Use duck_array_ops.ravel * Use duck_array_ops.reshape
* MRC -- Selecting with string for cftime See discussion in #9138 This commit and pull request mostly serves as a staging group for a potential fix. Test with: ``` pytest xarray/tests/test_cftimeindex.py::test_cftime_noleap_with_str ``` * effectively remove fastpath * Add docstring * Revert "effectively remove fastpath" This reverts commit 0f1a5a2. * Fix by reassigning coordinate * Update what's new entry * Simplify if condition --------- Co-authored-by: Spencer Clark <[email protected]>
* don't remove `netcdf4` from the upstream-dev environment * also stop removing `h5py` and `hdf5` * hard-code the precision (I believe this was missed in #9081) * don't remove `h5py` either * use on-diks _FillValue as standrd expects, use view instead of cast to prevent OverflowError. * whats-new * unpin `numpy` * rework UnsignedCoder * add test * Update xarray/coding/variables.py Co-authored-by: Justus Magin <[email protected]> --------- Co-authored-by: Kai Mühlbauer <[email protected]> Co-authored-by: Kai Mühlbauer <[email protected]> Co-authored-by: Deepak Cherian <[email protected]>
* don't remove `cf_units` (and thus `iris`) [skip-ci] * try keeping netcdf4, h5netcdf, and h5py
* `NaN` → `nan` * new numpy scalar repr * unpin `numpy` in the docs [skip-ci] * try extracting the scalars to pass to `linspace` * use `numpy` instead of `numpy.array_api` [skip-ci] and mention that we're falling back to `numpy.array_api` in `numpy<2.0` * Update ci/requirements/doc.yml * Remove 2D cross product doctests * One more fix --------- Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: Deepak Cherian <[email protected]> Co-authored-by: Deepak Cherian <[email protected]>
* Add a `.drop_attrs` method Part of #3891 * Add tests * Add explicit coords test * Use `._replace` for half the method * . * Add a `deep` kwarg (default `True`?) * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * api * Update xarray/core/dataarray.py Co-authored-by: Michael Niklas <[email protected]> * Update xarray/core/dataset.py Co-authored-by: Michael Niklas <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Michael Niklas <[email protected]>
This reverts commit 42fd510.
This reverts commit f85da8a.
Seems to require adding an exclude for `build` and an `__init__.py` file in the `properties` directory...
* Fix typing for test_plot.py * Update test_plot.py * make sure we actually get ndarrays here, I get it locally at least * Add a minimal test and ignore in real test * Update test_plot.py * Update test_plot.py
* allow per-variable choice of mask_and_scale in open_dataset * simplify docstring datatype * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * dict -> Mapping in type annotation Co-authored-by: Michael Niklas <[email protected]> * use typevar for _item_or_default annotation Otherwise you lose all typing when you use that because it returns Any. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * implement feature for 4 additional parameters * fix default value inconsistency * add what's new + None annotation * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * _item_or_default return type T | None * remove deault default value _item_or_default * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * docstring dtype naming --------- Co-authored-by: Mathijs Verhaegh <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Michael Niklas <[email protected]>
* remove pandas from ignore missing imports * add Any to dim arg of concat as placeholder * allow sequence of np.ndarrays as coords in dataArray constructor * fix several typing issues in tests * fix more types * more fixes * more typing... * we are getting there? * who might have guessed it... more typing * continue fixing typing issues * fix some typed_ops * fix last non-typed-ops errors * update typed ops * remove useless DaskArray type in scalar or array type * fix missing import in type_checking * fix import * improve cftime offsets typing * fix classvars * fix some checks * fix a broken test * improve typing of test_concat * fix broken concat * add whats-new --------- Co-authored-by: Maximilian Roos <[email protected]>
- Solve the numpy issues with appropriate ignores - Hide the NamedArray issues with blanket ignores
* Grouper, Resampler as public API * Add test * Add docs * Fix test * fix types. * bugfix * Better binning API * docs.fixes * Apply suggestions from code review Co-authored-by: Illviljan <[email protected]> * Fix typing * clean up reprs * Allow passing dicts * Apply suggestions from code review Co-authored-by: Justus Magin <[email protected]> * Update xarray/core/common.py Co-authored-by: Justus Magin <[email protected]> * Review comments * Fix docstring * Try to fix typing * Nicer error * Try fixing types * fix * Apply suggestions from code review Co-authored-by: Michael Niklas <[email protected]> * Review comments * Add whats-new note * Fix * Add more types * Fix link --------- Co-authored-by: Illviljan <[email protected]> Co-authored-by: Justus Magin <[email protected]> Co-authored-by: Michael Niklas <[email protected]>
Co-authored-by: Deepak Cherian <[email protected]>
* Remove `base`, `loffset` in Resampler * resample: Remove how/dim checks * lint * cleanups * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update test_cftimeindex_resample.py * cleanup --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* groupby, resample: Deprecate some positional args * Change version to 2024.07.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )