Skip to content

Refactoring ML features#748

Merged
babenek merged 37 commits intoSamsung:mainfrom
babenek:mlfeatnew
Dec 17, 2025
Merged

Refactoring ML features#748
babenek merged 37 commits intoSamsung:mainfrom
babenek:mlfeatnew

Conversation

@babenek
Copy link
Contributor

@babenek babenek commented Jul 30, 2025

Description

Please include a summary of the change and which is fixed.

  • reduce ML_HUNK to 64
  • transform Search ml features to float (centralized -1.0 ; 1.0)
  • split variable-based features for Id detect and others (AWS keys id are sensitive)

How has this been tested?

Please describe the tests that you ran to verify your changes.

  • UnitTest
  • Benchmark

@codecov-commenter
Copy link

codecov-commenter commented Jul 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.31%. Comparing base (a2cb912) to head (4d03399).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #748   +/-   ##
=======================================
  Coverage   92.31%   92.31%           
=======================================
  Files         142      142           
  Lines        5750     5750           
  Branches      933      933           
=======================================
  Hits         5308     5308           
  Misses        286      286           
  Partials      156      156           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@babenek babenek changed the title New ML features Refactoring ML features Dec 17, 2025
@babenek babenek marked this pull request as ready for review December 17, 2025 11:05
@babenek babenek requested a review from a team as a code owner December 17, 2025 11:05
@babenek babenek merged commit f646f46 into Samsung:main Dec 17, 2025
22 of 23 checks passed
@babenek babenek deleted the mlfeatnew branch December 17, 2025 14:28
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