Skip to content

Update OpenTelemetry dependencies in pyproject.toml - #7

Closed
masci wants to merge 1 commit into
mainfrom
masci-patch-1
Closed

Update OpenTelemetry dependencies in pyproject.toml#7
masci wants to merge 1 commit into
mainfrom
masci-patch-1

Conversation

@masci

@masci masci commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Removed version constraints for OpenTelemetry dependencies to make the package installable in Colab

Removed version constraints for OpenTelemetry dependencies.
@masci
masci marked this pull request as ready for review June 16, 2026 07:58
Copilot AI review requested due to automatic review settings June 16, 2026 07:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the project’s declared runtime dependencies by removing the minimum version constraints on the OpenTelemetry packages, with the goal of improving installability in environments like Google Colab.

Changes:

  • Removed >=1.20 lower bounds from opentelemetry-api, opentelemetry-sdk, and opentelemetry-exporter-otlp in pyproject.toml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pyproject.toml
Comment on lines +14 to +16
"opentelemetry-api",
"opentelemetry-sdk",
"opentelemetry-exporter-otlp",
@masci
masci marked this pull request as draft July 21, 2026 17:28
@milistu

milistu commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@masci Should we close this? I do not think it is relevant anymore

@masci

masci commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator Author

@masci Should we close this? I do not think it is relevant anymore

@milistu I still cannot install the package from Colab with !pip install git+https://github.com/elastic/elastic-evals-sdk-python.git

I get this error, related to how otel is pinned:

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
google-adk 2.7.0 requires opentelemetry-api<=1.43,>=1.39, but you have opentelemetry-api 1.44.0 which is incompatible.
google-adk 2.7.0 requires opentelemetry-sdk<=1.43,>=1.39, but you have opentelemetry-sdk 1.44.0 which is incompatible.
pyiceberg 0.11.1 requires rich<15.0.0,>=10.11.0, but you have rich 15.0.0 which is incompatible.
bigframes 2.42.0 requires rich<14,>=12.4.4, but you have rich 15.0.0 which is incompatible.

I think the framework should work seamlessly on Colab

@milistu

milistu commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

@masci Should we close this? I do not think it is relevant anymore

@milistu I still cannot install the package from Colab with !pip install git+https://github.com/elastic/elastic-evals-sdk-python.git

I get this error, related to how otel is pinned:

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
google-adk 2.7.0 requires opentelemetry-api<=1.43,>=1.39, but you have opentelemetry-api 1.44.0 which is incompatible.
google-adk 2.7.0 requires opentelemetry-sdk<=1.43,>=1.39, but you have opentelemetry-sdk 1.44.0 which is incompatible.
pyiceberg 0.11.1 requires rich<15.0.0,>=10.11.0, but you have rich 15.0.0 which is incompatible.
bigframes 2.42.0 requires rich<14,>=12.4.4, but you have rich 15.0.0 which is incompatible.

I think the framework should work seamlessly on Colab

Got it, I agree with you. I just thought that it was solved and the PR was left hanging. My mistake.
At first look it seems like our tight dependencies do not match with the existing packages pre-installed on Colab.
I will take a look

@masci

masci commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by #49

@masci masci closed this Aug 20, 2026
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.

3 participants