Skip to content

MS-124 Use decision policy to determine very good/good/fair matches#1277

Merged
luhmirin-s merged 1 commit into
mainfrom
feature/MS-124-matching-thresholds
Jul 23, 2025
Merged

MS-124 Use decision policy to determine very good/good/fair matches#1277
luhmirin-s merged 1 commit into
mainfrom
feature/MS-124-matching-thresholds

Conversation

@luhmirin-s
Copy link
Copy Markdown
Contributor

JIRA ticket
Will be released in: 2025.3.0

Notable changes

  • Use decision policy to count very good/good/fair matches on the match screen.
  • The hardcoded values were kept to have some kind of fallback, but realistically those should never be used.

Testing guidance

  • Setup reasonable decision policies per SDK (e.g. 20/50/90 for SimFace).
  • Run identifications with known and unknown faces - the amounts of very good/good matches should match the confidence band counts in the result.

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, meladRaouf and ybourgery and removed request for a team July 22, 2025 12:28
@cla-bot cla-bot Bot added the ... label Jul 22, 2025
@luhmirin-s luhmirin-s force-pushed the feature/MS-124-matching-thresholds branch from 5ca460f to 50bb620 Compare July 22, 2025 12:56
@luhmirin-s luhmirin-s force-pushed the feature/MS-124-matching-thresholds branch from 50bb620 to 945eef3 Compare July 22, 2025 13:17
@sonarqubecloud
Copy link
Copy Markdown

@luhmirin-s luhmirin-s merged commit 05d641b into main Jul 23, 2025
12 checks passed
@luhmirin-s luhmirin-s deleted the feature/MS-124-matching-thresholds branch July 23, 2025 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants