Skip to content

OpenAI Model Configuration Steps#88

Closed
teresaqhoang wants to merge 29 commits intomicrosoft:mainfrom
craigomatic:tehoang/embeddings-configuration
Closed

OpenAI Model Configuration Steps#88
teresaqhoang wants to merge 29 commits intomicrosoft:mainfrom
craigomatic:tehoang/embeddings-configuration

Conversation

@teresaqhoang
Copy link
Copy Markdown
Contributor

Motivation and Context

  1. Split setup in Sample 4 into two separate configuration steps for Completion and Embeddings models
  2. Hooked up oai and aoai APIs to load models into dropdown

Description

  1. new component ModelConfig that makes API call to load and populate models / deployments into dropdown using key and endpoint where applicable
  2. State changes to maintain values between tab transitions

image
image

Contribution Checklist

craigomatic and others added 29 commits March 13, 2023 16:42
Adding Relevance Slider and Send Button to QnA Chat
Remove IFileSystemConnector changes
Remove comment
1. Based on some feedback Shawn posted in another PR I think we should make CodeSkill into a sample project (GitHubExampleSkill) rather than publish it as a nuget package. I have the changes made for this.
2. Also change the messages in the UI to reflect what the sample is doing.
3. Fixes:
    - Spinner working
    - Disable download button while downloading
    - Add a useEffect to add the response to the ChatHistory
4. One experimental change which only summarises files above a certain file size. This saves us some API calls so the .md files can be summarised in ~30 secs (as opposed to ~60 secs)
…rror_msgs

Better error messages for sample 4
- Remove Code.Skill
- Add File label
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
…oft#17 (microsoft#88)

### Motivation and Context

Add ability to set Web App region in deployment - Fixes microsoft#17
 
### Description

This PR adds support to the `deploy-azure.ps1` and `deploy-azure.sh`
files to pass through the region to deploy the Static Web App to. This
will default to `westus2 and is set in the .PS1/SH files.

This enables deploying all the resources to the same region, even though
Static Web Apps is supported in only a few regions. The one region that
all resource types are supported in is WestEurope.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

Tests:
`azure-deploy.ps1`
<img width="845" alt="image"
src="https://github.com/microsoft/chat-copilot/assets/7589164/dfc1412a-0d7c-4045-adc8-c07f0202adc2">

<img width="944" alt="image"
src="https://github.com/microsoft/chat-copilot/assets/7589164/d6af1409-892a-44b4-a3c0-18783ae2170e">

`azure-deploy.sh`
<img width="1110" alt="image"
src="https://github.com/microsoft/chat-copilot/assets/7589164/4ef5abd8-6d33-4386-8254-b87af5f99aae">

<img width="969" alt="image"
src="https://github.com/microsoft/chat-copilot/assets/7589164/5291a3dc-6dd3-4fc2-87ea-0da8f143cd37">
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.

4 participants