Skip to content

Commit 8a8356a

Browse files
authored
fix: replace 'retrival' with 'retrieval' (#657)
1 parent 8ba4daa commit 8a8356a

File tree

7 files changed

+15
-15
lines changed

7 files changed

+15
-15
lines changed

backend/app/api/admin_routes/knowledge_base/graph/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ class KBKnowledgeGraphRetrievalConfig(BaseModel):
5050
class KBRetrieveKnowledgeGraphRequest(BaseModel):
5151
query: str
5252
llm_id: int
53-
retrival_config: KBKnowledgeGraphRetrievalConfig
53+
retrieval_config: KBKnowledgeGraphRetrievalConfig
5454

5555

5656
### Experimental

backend/app/api/admin_routes/knowledge_base/graph/routes.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@ def retrieve_kb_knowledge_graph(
207207
retriever = KnowledgeGraphSimpleRetriever(
208208
db_session=db_session,
209209
knowledge_base_id=kb_id,
210-
config=request.retrival_config.knowledge_graph,
210+
config=request.retrieval_config.knowledge_graph,
211211
)
212212
knowledge_graph = retriever.retrieve_knowledge_graph(request.query)
213213
return KnowledgeGraphRetrievalResult(

backend/app/api/routes/retrieve/models.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,28 +6,28 @@
66
KnowledgeGraphRetrieverConfig,
77
)
88
from app.rag.retrievers.chunk.schema import VectorSearchRetrieverConfig
9-
from app.rag.retrievers.multiple_knowledge_base import FusionRetrivalBaseConfig
9+
from app.rag.retrievers.multiple_knowledge_base import FusionRetrievalBaseConfig
1010

11-
# Chunks Retrival
11+
# Chunks retrieval
1212

1313

14-
class ChunkRetrievalConfig(FusionRetrivalBaseConfig):
14+
class ChunkRetrievalConfig(FusionRetrievalBaseConfig):
1515
full_documents: Optional[bool] = False
1616
vector_search: VectorSearchRetrieverConfig
1717

1818

19-
class ChunksRetrivalRequest(BaseModel):
19+
class ChunksRetrievalRequest(BaseModel):
2020
query: str
2121
retrieval_config: ChunkRetrievalConfig
2222

2323

24-
## Knowledge Graph Retrival
24+
## Knowledge Graph retrieval
2525

2626

27-
class KnowledgeGraphRetrievalConfig(FusionRetrivalBaseConfig):
27+
class KnowledgeGraphRetrievalConfig(FusionRetrievalBaseConfig):
2828
knowledge_graph: KnowledgeGraphRetrieverConfig
2929

3030

31-
class KnowledgeGraphRetrivalRequest(BaseModel):
31+
class KnowledgeGraphRetrievalRequest(BaseModel):
3232
query: str
3333
retrieval_config: KnowledgeGraphRetrievalConfig

backend/app/api/routes/retrieve/routes.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
from app.exceptions import KBNotFound
1515
from app.rag.retrievers.chunk.schema import ChunksRetrievalResult
1616
from app.rag.llms.resolver import get_llm_or_default
17-
from .models import ChunksRetrivalRequest, KnowledgeGraphRetrivalRequest
17+
from .models import ChunksRetrievalRequest, KnowledgeGraphRetrievalRequest
1818

1919
router = APIRouter()
2020
logger = logging.getLogger(__name__)
@@ -24,7 +24,7 @@
2424
def retrieve_chunks(
2525
db_session: SessionDep,
2626
user: CurrentSuperuserDep,
27-
request: ChunksRetrivalRequest,
27+
request: ChunksRetrievalRequest,
2828
) -> ChunksRetrievalResult:
2929
try:
3030
config = request.retrieval_config
@@ -49,7 +49,7 @@ def retrieve_chunks(
4949
def retrieve_knowledge_graph(
5050
db_session: SessionDep,
5151
user: CurrentSuperuserDep,
52-
request: KnowledgeGraphRetrivalRequest,
52+
request: KnowledgeGraphRetrievalRequest,
5353
) -> KnowledgeGraphRetrievalResult:
5454
try:
5555
config = request.retrieval_config

backend/app/rag/chat/config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ def get_fast_dspy_lm(self, session: Session) -> dspy.LM:
159159
llama_llm = self.get_fast_llama_llm(session)
160160
return get_dspy_lm_by_llama_llm(llama_llm)
161161

162-
# FIXME: Reranker top_n should be config in the retrival config.
162+
# FIXME: Reranker top_n should be config in the retrieval config.
163163
def get_reranker(
164164
self, session: Session, top_n: int = None
165165
) -> Optional[BaseNodePostprocessor]:

backend/app/rag/rerankers/resolver.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ def resolve_reranker(
8888
raise ValueError(f"Got unknown reranker provider: {provider}")
8989

9090

91-
# FIXME: Reranker top_n should be config in the retrival config.
91+
# FIXME: Reranker top_n should be config in the retrieval config.
9292
def get_default_reranker_model(
9393
session: Session, top_n: int = None
9494
) -> Optional[BaseNodePostprocessor]:

backend/app/rag/retrievers/multiple_knowledge_base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
from app.rag.llms.dspy import get_dspy_lm_by_llama_llm
2222

2323

24-
class FusionRetrivalBaseConfig(BaseModel):
24+
class FusionRetrievalBaseConfig(BaseModel):
2525
llm_id: Optional[int] = None
2626
knowledge_base_ids: List[int]
2727
use_query_decompose: Optional[bool] = None

0 commit comments

Comments
 (0)