Skip to content

Conversation

@Si-ra-kri
Copy link

Summary

Adds early validation to fail with a clear error when empty DataFrames
are passed to Report.run.

Motivation

Passing empty input data currently results in a ZeroDivisionError deep
inside legacy drift calculations, which is confusing for users.

Changes

  • Added input validation for empty DataFrames
  • Added a unit test covering this case

Scope

  • No API changes
  • No metric logic changes

@github-actions
Copy link

github-actions bot commented Feb 7, 2026

📚 Artifacts deployed to GitHub Pages: https://evidentlyai.github.io/evidently/ci/#pr-1820-improve-input-validation

@Si-ra-kri
Copy link
Author

All checks are green. Happy to make any adjustments if needed.

@DimaAmega
Copy link
Collaborator

DimaAmega commented Feb 11, 2026

Hi @Si-ra-kri

Thanks for contribution!
But you have an empty test tests/report/test_report_input_validation.py
Let's add a test case there and we will merge it

@Si-ra-kri
Copy link
Author

Added a test case and pushed updates. Please let me know if anything else is needed 🙂

@DimaAmega
Copy link
Collaborator

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.

2 participants