Skip to content

[GSK-3914] Fix RagasMetric outdated message #2062

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
Merged
merged 7 commits into from
Nov 18, 2024
Merged

[GSK-3914] Fix RagasMetric outdated message #2062

merged 7 commits into from
Nov 18, 2024

Conversation

@henchaves
Copy link
Member

@henchaves henchaves commented Oct 31, 2024 •
edited
Loading

Description

This PR updates the error message which was outdated when trying to execute RagasMetric without returning an AgentAnswer object.

Related Issue

#2055

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix
@henchaves henchaves requested a review from pierlj October 31, 2024 19:45
@linear
Copy link

linear bot commented Oct 31, 2024

GSK-3914 Fix RagasMetric outdated error message
@henchaves henchaves self-assigned this Oct 31, 2024
@henchaves henchaves requested a review from a team November 14, 2024 17:27
@henchaves henchaves mentioned this pull request Nov 14, 2024
Closed
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 18, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud
@kevinmessiaen kevinmessiaen merged commit 285fd6c into main Nov 18, 2024
17 checks passed
@kevinmessiaen kevinmessiaen deleted the feature/gsk-3914-fix-ragasmetric-outdated-error-message branch November 18, 2024 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@henchaves @kevinmessiaen