Skip to content

Conversation

@gorloffslava
Copy link

[Description of PR]

TODO:

  • Unit tests and/or doctests in docstrings
  • Tests pass locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • Docs build locally
  • GitHub Actions CI passes
  • Test coverage to 100% (Codecov passes)

dstansby and others added 26 commits September 1, 2024 09:53
* Add codec categories in doc contents

* Improve contents structure

* Put registry in API

* Add release note

* Move pcodec
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.2 → v0.6.3](astral-sh/ruff-pre-commit@v0.6.2...v0.6.3)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.3 → v0.6.4](astral-sh/ruff-pre-commit@v0.6.3...v0.6.4)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Bumps [pypa/cibuildwheel](https://github.com/pypa/cibuildwheel) from 2.20.0 to 2.21.0.
- [Release notes](https://github.com/pypa/cibuildwheel/releases)
- [Changelog](https://github.com/pypa/cibuildwheel/blob/main/docs/changelog.md)
- [Commits](pypa/cibuildwheel@v2.20.0...v2.21.0)

---
updated-dependencies:
- dependency-name: pypa/cibuildwheel
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.4 → v0.6.5](astral-sh/ruff-pre-commit@v0.6.4...v0.6.5)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Bumps [pypa/cibuildwheel](https://github.com/pypa/cibuildwheel) from 2.21.0 to 2.21.1.
- [Release notes](https://github.com/pypa/cibuildwheel/releases)
- [Changelog](https://github.com/pypa/cibuildwheel/blob/main/docs/changelog.md)
- [Commits](pypa/cibuildwheel@v2.21.0...v2.21.1)

---
updated-dependencies:
- dependency-name: pypa/cibuildwheel
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.5 → v0.6.7](astral-sh/ruff-pre-commit@v0.6.5...v0.6.7)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.7 → v0.6.8](astral-sh/ruff-pre-commit@v0.6.7...v0.6.8)

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

Add python-rc

Try label

Try again

* Only install pcodec on <3.13

* Update Python 3.13.0 to release version

Co-authored-by: jakirkham <[email protected]>

---------

Co-authored-by: jakirkham <[email protected]>
Bumps [pypa/cibuildwheel](https://github.com/pypa/cibuildwheel) from 2.21.1 to 2.21.2.
- [Release notes](https://github.com/pypa/cibuildwheel/releases)
- [Changelog](https://github.com/pypa/cibuildwheel/blob/main/docs/changelog.md)
- [Commits](pypa/cibuildwheel@v2.21.1...v2.21.2)

---
updated-dependencies:
- dependency-name: pypa/cibuildwheel
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: update pre-commit hooks

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.8 → v0.6.9](astral-sh/ruff-pre-commit@v0.6.8...v0.6.9)

* Fix pre-commit config

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: David Stansby <[email protected]>
* Update c-blosc to v1.26.1

* Add changelog entry
* Apply repo-review rule PP304

PP304: Sets the log level in pytest
log_cli_level should be set. This will allow logs to be displayed on failures.

* Apply repo-review rule PP305

PP305: Specifies xfail_strict
xfail_strict should be set. You can manually specify if a check should be strict when setting each xfail.

* Apply repo-review rules PP306, PP307, PP308

PP306: Specifies strict config
--strict-config should be in addopts = [...]. This forces an error if a config setting is misspelled.

PP307: Specifies strict markers
--strict-markers should be in addopts = [...]. This forces all markers to be specified in config, avoiding misspellings.

PP308: Specifies useful pytest summary
An explicit summary flag like -ra should be in addopts = [...] (print summary of all fails/errors).

* Apply ruff/pygrep-hooks rule PGH004

PGH004 Use specific rule codes when using `noqa`
PGH004 Use specific rule codes when using `ruff: noqa`

* Apply and enforce ruff rules (RUF)

Apply ruff rule RUF012:
RUF012 Mutable class attributes should be annotated with `typing.ClassVar`

Disable ruff rule RUF001

* Apply and enforce ruff/flake8-bugbear rules (B)

B007 Loop control variable not used within loop body

B028 No explicit `stacklevel` keyword argument found

B904 Within an `except` clause, raise exceptions with `raise ... from err` or
     `raise ... from None` to distinguish them from errors in exception handling

B905 `zip()` without an explicit `strict=` parameter

Co-authored-by: David Stansby <[email protected]>

* Apply and enforce ruff/isort rules (I)

* Apply repo-review rule PC100

PC100: Has pre-commit-hooks
Must have https://github.com/pre-commit/pre-commit-hooks repo in .pre-commit-config.yaml

* Enforce repo-review rules

* Apply ruff/flake8-implicit-str-concat rules (ISC)

ISC001 Implicitly concatenated string literals on one line

ISC003 Explicitly concatenated string should be implicitly concatenated

* Apply ruff/refurb rules (FURB)

FURB163 Prefer `math.log10`/`math.log2` over `math.log` with a redundant base

* Fix DeprecationWarning

	'pkgutil.find_loader' is deprecated and slated for removal
	in Python 3.14; use importlib.util.find_spec() instead

---------

Co-authored-by: David Stansby <[email protected]>
* Deprecate a couple ruff/pyupgrade rules (UP)

* Enforce ruff/flake8-pytest-style rules (PT)

* Apply ruff/flake8-pytest-style rule PT006

PT006 Wrong type passed to first argument of `@pytest.mark.parametrize`; expected `tuple`

* Apply ruff/flake8-pytest-style rule PT007

PT007 Wrong values type in `@pytest.mark.parametrize` expected `list` of `tuple`

* Enforce ruff/flake8-bugbear rules (B)

* Apply ruff/flake8-executable rule EXE001

EXE001 Shebang is present but file is not executable

* Enforce ruff/pygrep-hooks rules (PGH)

* Apply ruff/flake8-simplify rule SIM102

SIM102 Use a single `if` statement instead of nested `if` statements

* Apply ruff/flake8-simplify rule SIM105

SIM105 Use `contextlib.suppress(ImportError)` instead of `try`-`except`-`pass`

* Apply ruff/flake8-raise rule RSE102

RSE102 Unnecessary parentheses on raised exception

* Enforce ruff/flake8-raise rules (RSE)
* Use inplace subtract in Delta encoding

* ruff reformatting

* Add a comment explaining use of `subtract` for speed

---------

Co-authored-by: David Stansby <[email protected]>
* Update optimized `Delta` codec for `bool`

Previously `bool` just worked with `Delta`. However this was not
actually tested. The optimized version switched to `np.subtract` for
in-place computation, which works for other types. Though `bool` needs
special handling. Fortunately this can be done with `np.not_equal`,
which has the same behavior.

Also include a test for `bool` data to make sure this is handled
correctly going forward.

* Simplify type check

This is a bit more succinct and gets to the core point. Namely
`arr.dtype` determines this code path.

Also comparing directly to `bool` works here. It is a bit faster as well
since we need not construct an `np.dtype` object.

* Drop extraneous `0`s
updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/scientific-python/cookie: 2024.04.23 → 2024.08.19](scientific-python/cookie@2024.04.23...2024.08.19)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Apply ruff/flake8-pie rule PIE808

PIE808 Unnecessary `start` argument in `range`

* Enforce ruff/flake8-pie rules (PIE)
Bumps [pypa/cibuildwheel](https://github.com/pypa/cibuildwheel) from 2.21.2 to 2.21.3.
- [Release notes](https://github.com/pypa/cibuildwheel/releases)
- [Changelog](https://github.com/pypa/cibuildwheel/blob/main/docs/changelog.md)
- [Commits](pypa/cibuildwheel@v2.21.2...v2.21.3)

---
updated-dependencies:
- dependency-name: pypa/cibuildwheel
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@gorloffslava gorloffslava marked this pull request as ready for review October 16, 2024 05:15
@gorloffslava gorloffslava merged commit 7da0d32 into eugo-inc:main Oct 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants