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

fix: rc field support for RequestErrorResponse; feat: better __eq__ and __hash__ implementations where appropriate #143

Merged
merged 11 commits into from
Feb 17, 2024

Conversation

tazlin
Copy link
Member

@tazlin tazlin commented Feb 17, 2024

  • In response to feat: Error Return Codes AI-Horde#366 (Error Codes AI-Horde#149), changes the way error responses are coerced into RequestErrorResponse objects, fixing a very apparent issue surrounding this seen in the worker.
  • In preparation for some (hopefully) more intuitive handling of objects from the SDK (particularly JobPop responses et al), additional implementations for hashability (__eq__ and __hash__) have been added where appropriate to facilitate comparison and to allow using certain SDK objects as keys for
    • A direct result of this change is that numerous API objects now extend HordeAPIDataObject instead of pydantic.BaseModel directly.
  • dicts (or other mappings).
  • Brings the test json responses/payloads up to date with the production API as of 17/02/2024
  • Fixes a bug with the tests where the 2 per 1 second rule rate limiting rule was not respected.
  • Updates linters/formatters to the latest available and applies those rules.
  • Changes some of the docs to point to the examples on github rather than manually including them.

tazlin and others added 11 commits February 17, 2024 11:10
Updates the requirements on [horde-model-reference](https://github.com/Haidra-Org/horde-model-reference), [pytest](https://github.com/pytest-dev/pytest), [black](https://github.com/psf/black), [ruff](https://github.com/astral-sh/ruff) and [pre-commit](https://github.com/pre-commit/pre-commit) to permit the latest version.

Updates `horde-model-reference` to 0.5.4
- [Commits](Haidra-Org/horde-model-reference@v0.5.4...v0.5.4)

Updates `pytest` from 7.4.4 to 8.0.0
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@7.4.4...8.0.0)

Updates `black` from 23.12.1 to 24.2.0
- [Release notes](https://github.com/psf/black/releases)
- [Changelog](https://github.com/psf/black/blob/main/CHANGES.md)
- [Commits](psf/black@23.12.1...24.2.0)

Updates `ruff` from 0.1.14 to 0.2.1
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@v0.1.14...v0.2.1)

Updates `pre-commit` to 3.6.1
- [Release notes](https://github.com/pre-commit/pre-commit/releases)
- [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md)
- [Commits](pre-commit/pre-commit@v3.6.0...v3.6.1)

---
updated-dependencies:
- dependency-name: horde-model-reference
  dependency-type: direct:production
  dependency-group: python-packages
- dependency-name: pytest
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: python-packages
- dependency-name: black
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: python-packages
- dependency-name: ruff
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: python-packages
- dependency-name: pre-commit
  dependency-type: direct:development
  dependency-group: python-packages
...

Signed-off-by: dependabot[bot] <[email protected]>
@tazlin tazlin added the release:patch Version _._.x label Feb 17, 2024
@tazlin tazlin merged commit 43317ab into releases Feb 17, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:patch Version _._.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant