okirialbert commented on PR #38465:
URL: https://github.com/apache/airflow/pull/38465#issuecomment-2250428282
> @okirialbert Do you wish to continue with this change?
Yes I'll close it. I'll submit a PR for this with the required changes.
--
This is an automated message from the
kaxil commented on PR #38465:
URL: https://github.com/apache/airflow/pull/38465#issuecomment-2248851894
@okirialbert Do you wish to continue with this change?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
github-actions[bot] commented on PR #38465:
URL: https://github.com/apache/airflow/pull/38465#issuecomment-2239319321
This pull request has been automatically marked as stale because it has not
had recent activity. It will be closed in 5 days if no further activity occurs.
Thank you for
utkarsharma2 commented on PR #38465:
URL: https://github.com/apache/airflow/pull/38465#issuecomment-2123110819
Also, we need to make sure the systems tests are working with these changes
-
1.
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607938720
##
generated/provider_dependencies.json:
##
@@ -331,7 +331,8 @@
"cohere": {
"deps": [
"apache-airflow>=2.6.0",
- "cohere>=4.37,<5"
+
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607951700
##
airflow/providers/cohere/hooks/cohere.py:
##
@@ -46,23 +59,34 @@ def __init__(
conn_id: str = default_conn_name,
timeout: int | None =
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607944404
##
airflow/providers/cohere/hooks/cohere.py:
##
@@ -46,23 +59,34 @@ def __init__(
conn_id: str = default_conn_name,
timeout: int | None =
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607938720
##
generated/provider_dependencies.json:
##
@@ -331,7 +331,8 @@
"cohere": {
"deps": [
"apache-airflow>=2.6.0",
- "cohere>=4.37,<5"
+
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607936623
##
airflow/providers/cohere/operators/embedding.py:
##
@@ -58,11 +72,17 @@ def __init__(
self.input_text = input_text
self.timeout = timeout
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607935037
##
airflow/providers/cohere/operators/embedding.py:
##
@@ -39,6 +41,17 @@ class CohereEmbeddingOperator(BaseOperator):
information for Cohere. Defaults
utkarsharma2 commented on code in PR #38465:
URL: https://github.com/apache/airflow/pull/38465#discussion_r1607933073
##
airflow/providers/cohere/hooks/cohere.py:
##
@@ -46,23 +59,34 @@ def __init__(
conn_id: str = default_conn_name,
timeout: int | None =
potiuk commented on PR #38465:
URL: https://github.com/apache/airflow/pull/38465#issuecomment-2041603712
Seems that `cohere 5` forces installation of opentelemetry exporter . Can we
do something about it ?
```
#58 33.98 opentelemetry-exporter-otlp-proto-common 1.24.0 has
okirialbert opened a new pull request, #38465:
URL: https://github.com/apache/airflow/pull/38465
This PR adds support for Cohere 5.0.0 update
---
**^ Add meaningful description above**
Read the **[Pull Request
13 matches
Mail list logo