fix value error messages in data drift calculations #367
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tapot
deleted the
330-update_error_messages_for_no_values_case_in_drift_calculations
branch
emeli-dral
added a commit
that referenced
this pull request
Oct 26, 2022
* initial commit * Add Fisher's exact test * Replace MetricHtmlInfo by BaseWidgetInfo. Make id uuid by default. * New data drift metrics (#339) * rework data drift metrics * fix format and imports * fix notebooks * add empty check after data clean for drift + some refactoring * fix imports * add threshold for DatasetDriftMetric add tails in DatasetDriftMetric visual * refactor data drift * refactor data drift * add tests for DatasetDriftMetric * fix checks and titles for drift * fix style * update title in ColumnDriftMetric * implement columns for DatasetDriftMetric and DataDriftTable * fix data structure and json output for DataDriftTable * fix data structure and json output for DatasetDriftMetric * fix after main merge * fix with black * add reworked ColumnRegExpMetric (#348) * add reworked ColumnRegExpMetric * move ColumnRegExpMetric to a separate module, fix visual, add unittests * fix table in html view, update an example * fix ColumnRegExpMetric import in notebooks * fix notebook imports * add tabs for ColumnRegExpMetric * fix after main merge * fix after main merge * fix imports with isort * add anderson ksamp and its test * fix doc * fix description * added hellinger_distance for drift detection * isort * Delete index.js.LICENSE.txt * Delete index.js * Added some examples of metrics and metric presets usage Added some examples of tests and test presets usage Removed outdated example with metrics * move ColumnRegExpMetric data classes to the metric module (#360) * fix warning about duplicated columns in data drift (#361) * fix warning about duplicated columns in correlation calculation in data drift * make a new list, do not modify num_feature_names * Added the example of stattest specification for TestSuites * Update readme.md * Update readme.md * add anderson example in notebook * remove used features from wasserstein * fix anderson not found * check custom test * Update all-tests.md * Update run-tests.md * Update run-tests.md * Update README.md * Add files via upload * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update examples.md * Update examples.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * fix value error messages in data drift calculations (#367) * fix value error messages in data drift calculations * add error messages about missed column * Update missing values metrics (#357) * implement ColumnMissingValuesMetric and move DataIntegrityNullValuesMetrics to DatasetMissingValuesMetric * fix isort and black * fix notebook import and naming * fix isort + black * fix ColumnMissingValuesMetricRenderer and DatasetMissingValuesMetricRenderer * ass sort in ColumnMissingValuesMetric * fix ColumnMissingValuesMetric view * fix DatasetMissingValuesMetric view * some rename null values -> missed values * fix flake8 * add ColumnMissingValuesMetric unit tests * move DatasetMissingValuesMetric to a separate module * add test_dataset_missing_values_metrics_value_error * fix number_of_rows_with_nulls * fix labels texts * update notebook example * initial commit * Add Fisher's exact test * Update test_stattests.py * fix lint,sort * Fix contingency matrix boundary cases, and add tests * fix conflicts * fix fisher's exact test * fix mypy * fix black and remove checks Co-authored-by: Mert Bozkır <mert.bozkirr@gmail.com> Co-authored-by: Vyacheslav Morov <v.morov@corp.mail.ru> Co-authored-by: Tapot <novakche@yandex.ru> Co-authored-by: inderpreetsingh01 <inderpreetsinghchhabra23@gmail.com> Co-authored-by: inderpreetsingh01 <54892545+inderpreetsingh01@users.noreply.github.com> Co-authored-by: Emeli Dral <emeli.dral@gmail.com> Co-authored-by: elenasamuylova <67064421+elenasamuylova@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
get_one_column_drift(from Exception "division by 0" error with NaNs in DataDrift if stattest jensenshannon is used #330)test_prefix