Commits on Oct 23, 2024

1. [Number Series Copilot] Bug Fixes in Intent Understanding and Field V… …

   …alidation (#2111)
   
   <!-- Thank you for submitting a Pull Request. If you're new to
   contributing to BCApps please read our pull request guideline below
   * https://github.com/microsoft/BCApps/Contributing.md
   -->
   
   #### Summary <!-- Provide a general summary of your changes -->
   
   This PR addresses three critical bugs in the Number Series Copilot:
   
   1. **Bug 1: Incorrect Intent Mapping for "Generate" Command**
   - The system was misinterpreting the "generate" command, often preparing
   Number Series for the next period instead of creating a new one. The fix
   involves updating the tools definition and prompt to clarify the
   distinction between creating new Number Series and generating Number
   Series for the next period.
   
   2. **Bug 2: Incorrect Field Check for Existing Number Series**
   - The system was incorrectly performing checks on all fields for
   existing Number Series. The fix modifies the behavior to exclude
   pre-existing Number Series from the field validation checks, ensuring
   that only mandatory fields are validated for AI generated series.
   
   3. **Bug 3: Blank Fields Causing Validation Errors**
   - The system was generating Number Series without values for optional
   fields, causing validation errors. The fix refines the validation logic
   so that only mandatory fields are checked, allowing optional fields to
   remain blank if necessary.
   
   **Steps to Implement:**
   
   1. Refactor the LLM prompt and tools definition to improve the
   understanding of user intent between creating and preparing Number
   Series for different periods.
   2. Update the logic to exclude field checks for existing Number Series
   and restrict checks to mandatory fields.
   
   **Acceptance Criteria:**
   
   - Users can create Number Series without mistakenly triggering
   next-period Number Series generation when using the word "generate."
   - Existing Number Series are excluded from unnecessary checks, and only
   mandatory fields are validated for new series.
   
   These fixes aim to enhance the accuracy and reliability of the Number
   Series Copilot, ensuring it correctly interprets user commands and
   validates the necessary fields.
   
   
   #### Work Item(s) <!-- Add the issue number here after the #. The issue
   needs to be open and approved. Submitting PRs with no linked issues or
   unapproved issues is highly discouraged. -->
   
   Fixes #2106 
   
   Fixes
   [AB#550892](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/550892),
   [AB#544073](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/544073)

   

   DmitryKatson authored and JesperSchulz committed Oct 23, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 92833c9
   • Browse repository at this point

   Copy the full SHA

   92833c9 View commit details

   Browse the repository at this point in the history