Properties for diagnostic report #424
Merged
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
* definition * unit tests * integration test * move integration test * typo * typo 2 * docstring
* def 1 * def 2 + test * docstring * add test error * docstring
* definition * unit test * integration test * docstring * modify integration test * typo * quotes * address comments * fix lint * add test error * sample size * blank line * blank line
* coverage * boundary * synthesis * nanmean * moving integration tests * rebase
* first changes * before refactoring quality_report * refactor quality_report * print result diagnostic report * base init * integration test * fix test * docstring * docstring + get_results without verbose * check_report_generated * docstring * validate_categorical_values * remove out parameter * _validate_categorical_values
* define print method + _get_num_iterations * redefine base multi table property + lint * def base multi table report * tests base multi table (stable) * tests base multi table report * multi table diagnostic report def + tests * quality report def + int test + fix cardinality * unit tests * move base_report form single_table to report + uti * remove notebook * fix minimum version * fix lint * address comment 1 * change error message * details_property to details * modify get_num_iteration + address comment 2 * change _num_iteration_case names * remove _only_multi_table attribute * change error message cardinality * ind
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## master #424 +/- ##
==========================================
- Coverage 77.31% 76.33% -0.99%
==========================================
Files 83 91 +8
Lines 3394 3304 -90
==========================================
- Hits 2624 2522 -102
- Misses 770 782 +12
☔ View full report in Codecov by Sentry. |
* Ensure that the Synthesis property score doesn't change * adding integration tests
#438) * Adding class to handle printing of report results * Updating unit tests to match refactored code * Adding integration test for multiple generate calls * fixing lint * renaming test files to match actual files * making single_table.property details public
* Fix np.nan shown in results * Fix lint * Fix lint
* move to base report * use staticmethod
This was referenced Sep 13, 2023
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.
Resolve #394,
Resolve #395,
Resolve #398,
Resolve #400,
Resolve #409,
Resolve #411,
Resolve #418,
resolve #425,
Resolve #427,
Resolve #429,
Resolve #430,
Resolve #431,
Resolve #432,
Resolve #433