Chatgpt Agent Example #1539
Merged
Chatgpt Agent Example #1539
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 tasks
|
I am not sure whether should I provide the Dockerfile since the chatgpt agent hasn't been released in flytekit. |
pingsutw
reviewed
Mar 4, 2024
docs/contribute.md
Outdated
| @@ -331,7 +331,7 @@ documentation. | |||
| Verify that the code and documentation look as expected: | |||
|
|
|||
| - Learn about the documentation tools [here](https://docs.flyte.org/en/latest/community/contribute.html#documentation) | |||
| - Install the requirements by running `pip install -r docs-requirements.txt`. | |||
| - Install the requirements by running `pip install -r docs-requirements.in`. | |||
There was a problem hiding this comment.
When I am using python3.9 to develop, use pip install -r docs-requirements.txt will have the error below.
However, use pip install -r docs-requirements.in can solve the case.
There was a problem hiding this comment.
Would this be resolved by running pip-compile docs-requirements.in first, then pip install -r requirements.txt?
There was a problem hiding this comment.
In my case, the docs-requirements.txt is generated by python3.11.
I use python 3.9 and the error says that some modules required python>=3.10.
So I just install docs-requirements.in instead.
Do you mean
pip-compile docs-requirements.inpip install -r docs-requirements.txt
compile it again, and install it?
I think it might work, too, but installingdocs-requirements.inwill give us more flexibility to support all python versions, at least frompython3.8topython3.12.
There was a problem hiding this comment.
I think we will want to resolve this in a separate PR so it doesn't block merging the ChatGPT example and updates to the other agent examples. @Future-Outlier would you be able to undo this change in this branch and create a separate branch and PR for it?
There was a problem hiding this comment.
No problem, thank you！
neverett
requested changes
Mar 4, 2024
docs/contribute.md
Outdated
| @@ -331,7 +331,7 @@ documentation. | |||
| Verify that the code and documentation look as expected: | |||
|
|
|||
| - Learn about the documentation tools [here](https://docs.flyte.org/en/latest/community/contribute.html#documentation) | |||
| - Install the requirements by running `pip install -r docs-requirements.txt`. | |||
| - Install the requirements by running `pip install -r docs-requirements.in`. | |||
There was a problem hiding this comment.
Would this be resolved by running pip-compile docs-requirements.in first, then pip install -r requirements.txt?
examples/chatgpt_agent/chatgpt_agent/chatgpt_agent_example_usage.py
Outdated
Show resolved
Hide resolved
examples/chatgpt_agent/chatgpt_agent/chatgpt_agent_example_usage.py
Outdated
Show resolved
Hide resolved
examples/chatgpt_agent/chatgpt_agent/chatgpt_agent_example_usage.py
Outdated
Show resolved
Hide resolved
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
…ge.py Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Future-Outlier
force-pushed
the
chatgpt_agent_example
branch
from
1557f2b to
b9f28bf
Compare
neverett
approved these changes
Mar 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Docs link
Index: https://flyte--1539.org.readthedocs.build/projects/cookbook/en/1539/auto_examples/chatgpt_agent/index.html
example: https://flyte--1539.org.readthedocs.build/projects/cookbook/en/1539/auto_examples/chatgpt_agent/chatgpt_agent_example_usage.html
Screenshots
Tracking Issues
flyteorg/flyte#3936
Related PRs
flyteorg/flyte#4992