Skip to content

refacto reranker - #7

Merged
Ahmath-Gadji merged 2 commits into
devfrom
cleaning-reranker_embedder
Jul 4, 2025
Merged

refacto reranker#7
Ahmath-Gadji merged 2 commits into
devfrom
cleaning-reranker_embedder

Conversation

@Ahmath-Gadji

Copy link
Copy Markdown
Collaborator

No description provided.

@Ahmath-Gadji

Copy link
Copy Markdown
Collaborator Author

@EnjoyBacon7 ragatouille and fastembed-gpu uneeded now and are deleted

@Ahmath-Gadji
Ahmath-Gadji merged commit 29a7c73 into dev Jul 4, 2025
@Ahmath-Gadji
Ahmath-Gadji deleted the cleaning-reranker_embedder branch July 7, 2025 14:13
EnjoyBacon7 added a commit that referenced this pull request Mar 12, 2026
…s, validate file_id, guard copy_file overwrite (#1, #2, #7, #12, #13)
Matteovanypersele pushed a commit to Matteovanypersele/openrag that referenced this pull request Jul 6, 2026
…point mutations

linagora#5 Preset rename now migrates partition references atomically: PgPresetRepository.rename
   inserts the new-named row, repoints every partition that referenced the old name
   (partitions are a plain string ref with no FK), then drops the old row in one
   transaction. PresetService.update_preset rejects renaming onto an existing name.
   Previously a rename orphaned partitions on a preset name that no longer existed.

linagora#6 ModelEndpointService.update_model_endpoint evicts the 'default' alias client cache
   when the updated endpoint is the current default, so callers that resolved 'default'
   don't keep a stale client (mirrors set_default).

linagora#7 delete_model_endpoint promotes a surviving endpoint to default when the current
   default is deleted, so the 'default' alias keeps resolving instead of disappearing
   (load_all only adds it for an is_default row) and a later factory("default") raising.

Adds unit tests for all three plus a real-Postgres check of the rename migration.
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.

1 participant