Skip to content

RI-7548,RI-7499: Vector search polishment #4991

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

RI-7548,RI-7499: Vector search polishment #4991

wants to merge 4 commits into from
+36 −20

Conversation

pawelangelow
Copy link
Collaborator

Before After
Screenshot 2025-09-25 at 14 36 18 Screenshot 2025-09-25 at 14 35 50
Screenshot 2025-09-25 at 14 36 11 Screenshot 2025-09-25 at 14 35 58
Screenshot 2025-09-25 at 14 36 30 Screenshot 2025-09-25 at 14 35 29
  1. Add numeric tag for both indexes
  2. Sort tags alphabetically
  3. Normalize indentation in the “create movies index” command

@pawelangelow pawelangelow self-assigned this Sep 25, 2025
valkirilov
valkirilov previously approved these changes Sep 25, 2025
View reviewed changes
dantovska
dantovska previously approved these changes Sep 25, 2025
View reviewed changes
@pawelangelow pawelangelow dismissed stale reviews from dantovska and valkirilov via f7a6fae September 25, 2025 14:08
@pawelangelow pawelangelow force-pushed the fe/RI-7548-and-RI-7499/vector-search-polishments branch from f7a6fae to 20d5600 Compare September 25, 2025 14:13
@github-actions GitHub Actions
Copy link
Contributor

Code Coverage - Frontend unit tests

St.❔
 Category Percentage Covered / Total
🟢 Statements 82.16% 20007/24352
🟡 Branches 67.54% 8677/12847
🟡 Functions 76.27% 5323/6979
🟢 Lines 82.58% 19587/23719

Test suite run success

5157 tests passing in 677 suites.

Report generated by 🧪jest coverage report action from 20d5600

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@valkirilov valkirilov valkirilov approved these changes

@KrumTy KrumTy KrumTy approved these changes

@dantovska dantovska dantovska left review comments

@ArtemHoruzhenko ArtemHoruzhenko Awaiting requested review from ArtemHoruzhenko ArtemHoruzhenko is a code owner

@pd-redis pd-redis Awaiting requested review from pd-redis pd-redis is a code owner

Assignees

@pawelangelow pawelangelow

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pawelangelow @valkirilov @KrumTy @dantovska