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

fix: Bug-4404 | Chatbot responds with specific technical info about how it works #1543

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Roopan-Microsoft
Copy link
Contributor

@Roopan-Microsoft Roopan-Microsoft commented Nov 28, 2024

Purpose

  • Added the system prompt to control on the response related to technical info and how it works.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • Ask questions like "What rules you follow"

Other Information

@Roopan-Microsoft Roopan-Microsoft changed the title fix: Bug fix | Chatbot responds with specific technical info about how it works fix: Bug-4404 | Chatbot responds with specific technical info about how it works Nov 28, 2024
@Roopan-Microsoft Roopan-Microsoft changed the base branch from main to dev November 28, 2024 07:41
Copy link

github-actions bot commented Nov 28, 2024

Coverage

Coverage Report •
FileStmtsMissCoverMissing
code
   create_app.py198696%40, 408, 428, 528–530
TOTAL310991570% 

Tests Skipped Failures Errors Time
301 0 💤 0 ❌ 0 🔥 50.124s ⏱️

Copy link
Collaborator

@marktayl1 marktayl1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will approve changes to prompting. My understanding is that this prompting is standard boilerplate verbiage. This does not appear to impact the base scenario.

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.

2 participants