Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

Sync with main #23

Open
wants to merge 368 commits into
base: release
Choose a base branch
from
Open

Sync with main #23

wants to merge 368 commits into from

Conversation

vaibhavjainwiz
Copy link
Member

What this PR does / why we need it:

Special notes for your reviewer:

If applicable:

  • this PR contains documentation
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

mynhardtburger and others added 30 commits March 4, 2024 20:19
Signed-off-by: Mynhardt Burger <[email protected]>
Signed-off-by: Mynhardt Burger <[email protected]>
Add input token count to embedding, reranker, sentence similarity
Bumps [ruff](https://github.com/astral-sh/ruff) from 0.3.0 to 0.3.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.3.0...v0.3.1)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [ruff](https://github.com/astral-sh/ruff) from 0.3.1 to 0.3.2.
- [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.3.1...v0.3.2)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Mynhardt Burger <[email protected]>
Signed-off-by: Mynhardt Burger <[email protected]>
Signed-off-by: Mynhardt Burger <[email protected]>
Updates the requirements on [protobuf](https://github.com/protocolbuffers/protobuf) to permit the latest version.
- [Release notes](https://github.com/protocolbuffers/protobuf/releases)
- [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/protobuf_release.bzl)
- [Commits](protocolbuffers/protobuf@v3.19.0...v5.26.0)

---
updated-dependencies:
- dependency-name: protobuf
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Also scope connections to include the port in the map so multiple
connections can target the same hostname (e.g. localhost)

caikit#680

Signed-off-by: Gabe Goodhart <[email protected]>
Bumps [ruff](https://github.com/astral-sh/ruff) from 0.3.2 to 0.3.3.
- [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.3.2...v0.3.3)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Updates the requirements on [docstring-parser](https://github.com/rr-/docstring_parser) to permit the latest version.
- [Changelog](https://github.com/rr-/docstring_parser/blob/master/CHANGELOG.md)
- [Commits](rr-/docstring_parser@0.14.1...0.16)

---
updated-dependencies:
- dependency-name: docstring-parser
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Both default to False for compatibility

caikit#685

Signed-off-by: Gabe Goodhart <[email protected]>
The python protoc generator is not smart enough to not use . in python
module names.

caikit#685

Signed-off-by: Gabe Goodhart <[email protected]>
…tion

The protos have global state, unfortunately, so the tests need to be a
little less pedantic about expecting an exact set of protos and instead
just verify that the expected protos are there.

caikit#685

Signed-off-by: Gabe Goodhart <[email protected]>
gabe-l-hart and others added 30 commits June 21, 2024 15:51
One of the big follow ups I'd like to do from this PR is to create an
actual RuntimeServerContextType class that does the shimming between grpc
and fastapi so that we don't need to keep writing optional dependency
checks and type checks. If we do this, I think this would be a fine place
to reinstate this request_id logic so that the id becomes available to any
place downstream that wants to have it for logging purposes.

As it stands, though, the only use of this request_id is to attach it to
the trace which is useless because it will always have a unique ID for the
span anyway.

Signed-off-by: Gabe Goodhart <[email protected]>
…kit-global_predict_servicer-level

first version showing how otel compliant tracing can be enabled
Use the write_modules_json flag
Signed-off-by: gkumbhat <[email protected]>
Signed-off-by: Evaline Ju <[email protected]>
Co-authored-by: Gabe Goodhart <[email protected]>
Signed-off-by: Gaurav Kumbhat <[email protected]>
🐛  Bug fix for empty string iterable getting converted into iterable with None for predict calls
Signed-off-by: gkumbhat <[email protected]>
Signed-off-by: Michael Honaker <[email protected]>
Update Pydantic  Wrapper to support Pydantic 2.8
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.