[pull] main from xarray-contrib:main by pull[bot] · Pull Request #69 · jukent/cf-xarray

pull · 2022-04-28T06:45:05Z

See Commits and Changes for more details.

Can you help keep this open source service alive? 💖 Please sponsor : )

Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* check attrs before printing * explicit skip axes and coords * better var names * use one if * apply review and add what's new

On some instance of E-OBS dataset, the "units" is Celsius instead of the standard "celsius".

* import from the new locations and fall back to the old ones * use standard definition syntax instead

updates: - [github.com/psf/black: 22.3.0 → 22.6.0](psf/black@22.3.0...22.6.0) - [github.com/pre-commit/mirrors-mypy: v0.942 → v0.961](pre-commit/mirrors-mypy@v0.942...v0.961) - [github.com/myint/rstcheck: 3f92957478422df87bd730abde66f089cc1ee19b → v6.0.0.post1](rstcheck/rstcheck@3f92957...v6.0.0.post1) - [github.com/asottile/pyupgrade: v2.31.1 → v2.34.0](asottile/pyupgrade@v2.31.1...v2.34.0) - [github.com/pre-commit/pre-commit-hooks: v4.1.0 → v4.3.0](pre-commit/pre-commit-hooks@v4.1.0...v4.3.0) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

* Test with flox * Actually test * [test-upstream] * [test-upstream] * [test-upstream] * Fix grouped subtraction test a bit

…ow copies (#358)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Relax check_unchanged to adapt to upstream changes * [test-upstream] Co-authored-by: dcherian <deepak@cherian.net>

Bumps [xarray-contrib/ci-trigger](https://github.com/xarray-contrib/ci-trigger) from 1.1 to 1.2. - [Release notes](https://github.com/xarray-contrib/ci-trigger/releases) - [Commits](xarray-contrib/ci-trigger@v1.1...v1.2) --- updated-dependencies: - dependency-name: xarray-contrib/ci-trigger dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [mamba-org/provision-with-micromamba](https://github.com/mamba-org/provision-with-micromamba) from 13 to 14. - [Release notes](https://github.com/mamba-org/provision-with-micromamba/releases) - [Commits](mamba-org/provision-with-micromamba@a319a81...f347426) --- updated-dependencies: - dependency-name: mamba-org/provision-with-micromamba dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/psf/black: 22.6.0 → 22.8.0](psf/black@22.6.0...22.8.0) - [github.com/PyCQA/flake8: 4.0.1 → 5.0.4](PyCQA/flake8@4.0.1...5.0.4) - [github.com/pre-commit/mirrors-mypy: v0.961 → v0.981](pre-commit/mirrors-mypy@v0.961...v0.981) - [github.com/rstcheck/rstcheck: v5.0.0 → v6.1.0](rstcheck/rstcheck@v5.0.0...v6.1.0) - [github.com/executablebooks/mdformat: 0.7.14 → 0.7.16](hukkin/mdformat@0.7.14...0.7.16) - [github.com/nbQA-dev/nbQA: 1.3.1 → 1.5.2](nbQA-dev/nbQA@1.3.1...1.5.2) - [github.com/asottile/pyupgrade: v2.34.0 → v2.38.2](asottile/pyupgrade@v2.34.0...v2.38.2) - [github.com/keewis/blackdoc: v0.3.4 → v0.3.7](keewis/blackdoc@v0.3.4...v0.3.7) * Update .pre-commit-config.yaml Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Pascal Bourgault <bourgault.pascal@ouranos.ca> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…ed (#622) Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

The per-call ``from regex import match`` (with ``re`` fallback) inside ``_get_custom_criteria`` goes through ``importlib._bootstrap._find_and_load`` on every invocation, even when the module is already in ``sys.modules``. Concurrent threads calling cf accessors then serialize on the import lock, which shows up as large ``acquire`` time in py-spy profiles of multi-threaded users (e.g. async webservers serving many variables). Moving the import to module scope makes subsequent references a fast ``LOAD_GLOBAL`` with no lock acquisition. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.6.3 to 2.7.0. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](urllib3/urllib3@2.6.3...2.7.0) --- updated-dependencies: - dependency-name: urllib3 dependency-version: 2.7.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

The string is typically shared across all variables in a dataset, so parse once per distinct attr. Wrap the cached result in `Frozen` to prevent accidental mutation of the shared value. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Skip the per-key _get_all fan-out when every key is already a plain data-variable name — the slow path would just fall through to the same result. Decoration via get_associated_variable_names is unchanged. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

drop_bounds is a no-op unless the caller passed a scalar key, so list-key calls (ds.cf[[name]]) never need it. Inline the lookup so list-key paths skip the full-dataset bounds scan entirely. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

malmans2 and others added 5 commits April 27, 2022 08:33

Dependabot (#332)

735dec5

Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Bump actions/setup-python from 1 to 3 (#333)

0b2c055

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump codecov/codecov-action from 1 to 3 (#334)

29d14da

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump actions/checkout from 2 to 3 (#335)

5dd53ea

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Check attrs before printing guess (#331)

2c3801d

* check attrs before printing * explicit skip axes and coords * better var names * use one if * apply review and add what's new

pull Bot added the ⤵️ pull label Apr 28, 2022

abel-bzz and others added 24 commits May 3, 2022 06:23

ENH: Add "Celsius" alias (#336)

f24c1c3

On some instance of E-OBS dataset, the "units" is Celsius instead of the standard "celsius".

Explicit support for curvefit (#337)

1fbc074

Infer 1D bounds for nD variables. (#328)

33ce60b

keep compatibility with the development version of pint (#341)

ef7534d

* import from the new locations and fall back to the old ones * use standard definition syntax instead

Drop py37 (#296)

8c781ee

CI improvements (#342)

aac9a15

Explicit xarray imports (#343)

b031b45

Add howfairis badge. (#308)

77d6b68

rstcheck with sphinx config (#348)

1f59021

Bump actions/setup-python from 3 to 4 (#344)

b1fe3ff

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Avoid quadratic behaviour when dropping bounds (#349)

3ddf96a

Test with flox (#339)

2b889f4

* Test with flox * Actually test * [test-upstream] * [test-upstream] * [test-upstream] * Fix grouped subtraction test a bit

Print variable name for cell measures warnings (#350)

5138806

Release v0.7.4 (#351)

ae58327

Fix typo (#353)

5034a07

cf_xarray.datasets: hide auxiliary variables (#355)

d3b75a2

Performance improvement: Convert unnecessary (?) deep copies to shall…

4748033

…ow copies (#358)

Accounting for NoneType coordinates in attrs/encoding (#359)

92ae91c

Bump mamba-org/provision-with-micromamba from 12 to 13 (#364)

6994625

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Replace id comparison by value comparison in tests (#373)

a97ad1b

* Relax check_unchanged to adapt to upstream changes * [test-upstream] Co-authored-by: dcherian <deepak@cherian.net>

dependabot Bot and others added 21 commits February 3, 2026 12:27

Bump starlette from 0.47.3 to 0.49.1 (#616)

955e78b

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump urllib3 from 2.5.0 to 2.6.3 (#617)

a2fc7cc

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump fonttools from 4.59.2 to 4.60.2 (#615)

dd22c0c

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump pillow from 11.3.0 to 12.1.1 (#618)

b39d00c

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump requests from 2.32.5 to 2.33.0 (#624)

017306f

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Avoid overlapping bounds and/or gaps when points are irregularly spac…

9803061

…ed (#622) Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Bump actions/download-artifact from 7 to 8 (#620)

aede27a

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump actions/upload-artifact from 6 to 7 (#619)

982d8f6

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Fix link to cosima model agnostic analysis (#626)

56a6ba6

Fix mypy checks (#630)

5d05556

[pre-commit.ci] pre-commit autoupdate (#631)

24823dc

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Bump pytest from 8.4.2 to 9.0.3 (#634)

cc071e9

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump pygments from 2.19.2 to 2.20.0 (#628)

7bd3c7a

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump lxml from 6.0.1 to 6.1.0 (#635)

aa5790b

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Bump pillow from 12.1.1 to 12.2.0 (#633)

6c1144f

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump pypa/gh-action-pypi-publish from 1.13.0 to 1.14.0 (#636)

9d9dd89

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Bump codecov/codecov-action from 5.5.2 to 6.0.0 (#629)

07532e8

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump tornado from 6.5.2 to 6.5.5 (#621)

4205e7f

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Optimize add_bounds (#627)

4c0957c

Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>

Speed up Dataset.cf.bounds with a single-pass scan (#637)

8fa8978

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

dcherian force-pushed the main branch from 76a9b84 to 8fa8978 Compare May 2, 2026 03:56

dependabot Bot and others added 8 commits May 12, 2026 10:44

Memoize parse_cell_methods_attr

af9fb12

The string is typically shared across all variables in a dataset, so parse once per distinct attr. Wrap the cached result in `Frozen` to prevent accidental mutation of the shared value. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Cache parse_grid_mapping too

e27668a

Cleanup

0fd08cd

Update type

192f107

Include topology vars when subsetting with .cf (#640)

87fa240

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[pull] main from xarray-contrib:main#69

[pull] main from xarray-contrib:main#69
pull[bot] wants to merge 240 commits into
jukent:mainfrom
xarray-contrib:main

pull Bot commented Apr 28, 2022 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Conversation

pull Bot commented Apr 28, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

pull Bot commented Apr 28, 2022 •

edited

Loading