Skip to content

Conversation

@rhlbhatnagar
Copy link
Contributor

@rhlbhatnagar rhlbhatnagar commented Sep 28, 2025

No description provided.

@rhlbhatnagar rhlbhatnagar marked this pull request as draft September 28, 2025 21:49
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Sep 28, 2025
Comment on lines 18 to 19
if t.TYPE_CHECKING:
from langchain_core.callbacks.base import Callbacks
pass
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this?

# DIRECT PROMPT TEMPLATES (No PydanticPrompt dependencies)
# ============================================================================

CORRECTNESS_CLASSIFIER_PROMPT = """Given a ground truth and an answer statements, analyze each statement and classify them in one of the following categories: TP (true positive): statements that are present in answer that are also directly supported by the one or more statements in ground truth, FP (false positive): statements present in the answer but not directly supported by any statement in ground truth, FN (false negative): statements found in the ground truth but not present in answer. Each statement can only belong to one of the categories. Provide a reason for each classification.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These prompts should be in the "prompts" folder and not inline here

)
examples_text += "-----------------------------\n"

return f"""Decompose and break down each of the input sentences into one or more standalone statements. Each statement should be a standalone claim that can be independently verified.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move the prompts to it's own director and use .format to fill in the templates or wrap in a function there

@NirantK NirantK changed the title Mirgate Faithfulness + dependent metrics Migrate Faithfulness + dependent metrics Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants