[PR #16530] Update similarIssues.yml to have a lower tolerance #30938

Open
opened 2026-01-31 09:43:56 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/16530

State: closed
Merged: Yes


Summary of the Pull Request

The tolerance value for a similar issue was changed from 0.8 to 0.75.

References and Relevant Issues

This is because I changed the backend service for this to use pinecone instead of Azure AI search (see here f72fa59e23 ) and the metric changed as a result of that. They are slightly lower than they were before, so this should offset that.

Detailed Description of the Pull Request / Additional comments

Small PR so only one change of a value.

Validation Steps Performed

Validated with the same calls to the Azure AI search and then to the new database to determine that minusing 0.05 would be sufficient.

PR Checklist

  • No closed relevant issues
  • No specific Terminal tests
  • No need for documentation updates
  • No need for schema updates
**Original Pull Request:** https://github.com/microsoft/terminal/pull/16530 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request The tolerance value for a similar issue was changed from 0.8 to 0.75. ## References and Relevant Issues This is because I changed the backend service for this to use pinecone instead of Azure AI search (see here https://github.com/craigloewen-msft/GitGudIssues/commit/f72fa59e23c0b7501b613daea95371cb13831d42 ) and the metric changed as a result of that. They are slightly lower than they were before, so this should offset that. ## Detailed Description of the Pull Request / Additional comments Small PR so only one change of a value. ## Validation Steps Performed Validated with the same calls to the Azure AI search and then to the new database to determine that minusing 0.05 would be sufficient. ## PR Checklist - [X] No closed relevant issues - [X] No specific Terminal tests - [X] No need for documentation updates - [X] No need for schema updates
claunia added the pull-request label 2026-01-31 09:43:56 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30938