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

[Prompty] Support model config in prompty #2728

Merged
merged 14 commits into from
Apr 12, 2024
Merged

Conversation

lalala123123
Copy link
Contributor

@lalala123123 lalala123123 commented Apr 10, 2024

Description

  • prompty with azure openai
---
name: Basic Prompt
description: A basic prompt that uses the GPT-3 chat API to answer questions
model:
    api: chat
    configuration:
      type: azure_openai
      azure_deployment: gpt-35-turbo
      azure_endpoint: ${env:AZURE_ENDPOINT}
      api_key: ${env:AZURE_API_KEY}
    parameters:
      max_tokens: 128
      temperature: 0.2
inputs:
  firstName:
    type: string
    default: John
  lastName:
    type: string
    default: Doh
  question:
    type: string
---
system:
You are an AI assistant who helps people find information.
Use their name to address them in your responses.

user:
{{question}}

  • prompty with connection
---
name: Basic Prompt
description: A basic prompt that uses the GPT-3 chat API to answer questions
model:
    api: chat
    configuration:
      connection: azure_open_ai_connection
      type: azure_openai
      azure_deployment: gpt-35-turbo
    parameters:
      max_tokens: 128
      temperature: 0.2
inputs:
  firstName:
    type: string
    default: John
  lastName:
    type: string
    default: Doh
  question:
    type: string
---
system:
You are an AI assistant who helps people find information.
Use their name to address them in your responses.

user:
{{question}}

  • prompty with openai
---
name: Basic Prompt
description: A basic prompt that uses the GPT-3 chat API to answer questions
model:
    api: chat
    configuration:
      connection: azure_open_ai_connection
      type: openai
      model: gpt-35-turbo
      api_key: ${env:API_KEY}
      base_url: ${env:BASE_URL}
    parameters:
      max_tokens: 128
      temperature: 0.2
inputs:
  firstName:
    type: string
    default: John
  lastName:
    type: string
    default: Doh
  question:
    type: string
---
system:
You are an AI assistant who helps people find information.
Use their name to address them in your responses.

user:
{{question}}

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

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.

Copy link

github-actions bot commented Apr 10, 2024

SDK CLI Global Config Test Result zhrua/prompty_model_config

3 tests   3 ✅  1m 6s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit d8220d0.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Apr 10, 2024

promptflow SDK CLI Azure E2E Test Result zhrua/prompty_model_config

  4 files    4 suites   4m 15s ⏱️
218 tests 190 ✅  28 💤 0 ❌
872 runs  760 ✅ 112 💤 0 ❌

Results for commit d8220d0.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Apr 10, 2024

Executor Unit Test Result zhrua/prompty_model_config

778 tests   778 ✅  3m 42s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit d8220d0.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Apr 10, 2024

Executor E2E Test Result zhrua/prompty_model_config

238 tests   236 ✅  6m 53s ⏱️
  1 suites    2 💤
  1 files      0 ❌

Results for commit d8220d0.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Apr 10, 2024

SDK CLI Test Result zhrua/prompty_model_config

    4 files      4 suites   47m 19s ⏱️
  587 tests   565 ✅ 22 💤 0 ❌
2 348 runs  2 260 ✅ 88 💤 0 ❌

Results for commit d8220d0.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Apr 10, 2024

promptflow-core test result

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit d8220d0.

♻️ This comment has been updated with latest results.

wangchao1230
wangchao1230 previously approved these changes Apr 12, 2024
@wangchao1230 wangchao1230 merged commit 15960bf into main Apr 12, 2024
52 checks passed
@wangchao1230 wangchao1230 deleted the zhrua/prompty_model_config branch April 12, 2024 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants