Skip to content

Commit cbdb0d2

Browse files
mhordynskiCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 2195352 commit cbdb0d2

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

packages/ragbits-conversations/src/ragbits/conversations/piepline/examples/example1.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
async def main() -> None:
1010
"""
11-
Example of using convcrsation pipeline
11+
Example of using conversation pipeline
1212
"""
1313
llm = LiteLLM("gpt-4o")
1414
pipeline = ConversationPiepline(llm)

packages/ragbits-conversations/src/ragbits/conversations/piepline/state.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class ConversationPipelineState(BaseModel):
1414

1515
user_question: str
1616
session_id: str | None = None
17-
client_metada: dict[str, Any] = {}
17+
client_metadata: dict[str, Any] = {}
1818
history: ChatFormat = []
1919
rag_context: list[str] = []
2020

packages/ragbits-conversations/src/ragbits/conversations/piepline/steps.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ async def run(self, state: ConversationPipelineState) -> ConversationPipelineSta
6161
class HistoryCompressionStep(Step):
6262
"""
6363
A plugin that removes history from the conversation pipeline state
64-
and adds any nesseseaty context from history to the user question.
64+
and adds any necessary context from history to the user question.
6565
"""
6666

6767
def __init__(self, llm: LLM, compressor: ConversationHistoryCompressor | None = None) -> None:

packages/ragbits-core/src/ragbits/core/pipelines/pipeline.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,8 @@ def _validate_intrastep_types(self) -> None:
4747
"""
4848
Validate that the output type of each step is compatible with the input type of the next step.
4949
50-
Doesn't validate the first step input type and the last step output type because those are not know
51-
durint the initialization of the pipeline (due to the generic types and the specifics
50+
Doesn't validate the first step input type and the last step output type because those are not known
51+
during the initialization of the pipeline (due to the generic types and the specifics
5252
of the Python typing system).
5353
"""
5454
for i, step in enumerate(self.steps):
@@ -63,7 +63,7 @@ def _validate_intrastep_types(self) -> None:
6363
if not self._issubtype(previous_output_type, input_type):
6464
raise TypeError(
6565
f"Step {i} input type {input_type} is not compatible with "
66-
"previous step output type {previous_output_type}"
66+
f"previous step output type {previous_output_type}"
6767
)
6868

6969
def _validate_input_output_types(self, input_type: Type, output_type: Type) -> None:

0 commit comments

Comments
 (0)