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

[Sample] Add a sample which converts a langchain app to flex flow with custom connection #3090

Merged
merged 14 commits into from May 8, 2024

Conversation

D-W-
Copy link
Contributor

@D-W- D-W- commented Apr 30, 2024

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

This pull request introduces a new feature to the langchain-eval example in the flex-flows directory. The new feature includes a workflow for evaluating language chains, as well as changes to the README, data, and requirements files. The most significant changes are:

  1. A new GitHub workflow file .github/workflows/samples_flexflows_langchaineval_langchaineval.yml was added. This workflow is scheduled to run daily and on pull requests that modify specific paths. It sets up a Python 3.9 environment, installs requirements, and runs a notebook test, among other steps.

  2. The examples/README.md file was updated to include a link to the langchain-eval.ipynb notebook and its corresponding workflow badge.

  3. Two new data entries were added to the examples/flex-flows/langchain-eval/data.jsonl file.

  4. The examples/flex-flows/langchain-eval/flow.flex.yaml file was added, which contains the schema and entry point for the new feature.

  5. A new Python file examples/flex-flows/langchain-eval/flow.py was added. This file contains the LangChainEvaluator class, which is used to evaluate language chains.

  6. The examples/flex-flows/langchain-eval/langchain-eval.ipynb notebook was added. This notebook provides a tutorial on how to use the new feature.

  7. The examples/flex-flows/langchain-eval/requirements.txt file was updated to include the promptflow and langchain packages, as well as the python-dotenv package.

All Promptflow Contribution checklist:

  • The pull request does not introduce [breaking changes].
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@D-W- D-W- requested a review from a team as a code owner April 30, 2024 09:33
@github-actions github-actions bot added examples Improvements on examples fundamental labels Apr 30, 2024
elliotzh
elliotzh previously approved these changes Apr 30, 2024
@D-W- D-W- requested a review from a team as a code owner May 7, 2024 06:15
brynn-code
brynn-code previously approved these changes May 7, 2024
wangchao1230
wangchao1230 previously approved these changes May 7, 2024
@D-W- D-W- merged commit 5457e5f into main May 8, 2024
14 checks passed
@D-W- D-W- deleted the wanhan/custom_connection_sample branch May 8, 2024 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Improvements on examples fundamental
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants