Skip to content
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.

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

Update numerical_data_drift_metrics.py #865

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yogeshhh1207
Copy link

fixed #864, successfully added both the other two parameters in the function.

fixed Azure#864, successfully added both the other two parameters in the function.
@yogeshhh1207 yogeshhh1207 requested a review from a team as a code owner July 10, 2023 20:03
@yetamsft yetamsft added the safe to test Pull request from a fork that's been reviewed and is cleared for additional testing label Jul 11, 2023
@yetamsft yetamsft temporarily deployed to Testing July 11, 2023 17:48 — with GitHub Actions Inactive
@yetamsft yetamsft temporarily deployed to Testing July 11, 2023 17:48 — with GitHub Actions Inactive
@yetamsft
Copy link
Member

I merged a fix for this in this PR #875.

@elesel elesel force-pushed the main branch 3 times, most recently from 16dd412 to f103d2e Compare April 23, 2024 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Pull request from a fork that's been reviewed and is cleared for additional testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Required parameters missing when calling function _ks2sample_pandas_impl
2 participants