Repository navigation
Conversation
…er and vendor rtichoke_viz v0.22.0 - Vendor rtichoke_viz v0.22.0 release asset (SHA-256 64087fe0284ab2beb6e664dd419504a56000e217eaf403f33559896b67fff94d). - Implement _prediction_distribution_v2_spec_from_performance_data in _viz_spec_v2.py to construct canonical v0.22.0 PredictionDistributionSpec objects. - Implement standalone public create_probs_histogram() function in probs_distribution.py and export in rtichoke package namespace. - Update RtichokeBrowserChart to support single-file self-contained HTML rendering for prediction distribution charts. - Add deterministic unit tests and Playwright real-browser acceptance tests. Co-authored-by: uriahf <11351434+uriahf@users.noreply.github.com>
|
|
Thanks — the overall implementation, v0.22.0 adoption, public API, join keys, Before merging, please make one architectural correction so this consumer can
The lowest-level shared builder should accept already-prepared inputs, along It must only map those producer-owned quantities into the canonical The public standalone path may then use a convenience wrapper that calls: and passes their outputs to the shared prepared-data builder. The current function is named Do not change statistical behavior or the public create_probs_histogram() API.
Replace: with the established package terminology: Risk Percentile may remain as the user-facing explanatory label.
Serialize true_positives, true_negatives, false_positives, and false_negatives Add focused tests proving:
Keep Summary Report integration out of this PR. Do not merge yet. |
…ndardize confusion matrix integer serialization - Refactor _prediction_distribution_v2_spec to accept pre-computed distribution_data, performance_data, and evaluation_metadata without calling statistical producers. - Retain _prediction_distribution_v2_spec_from_performance_data as a convenience wrapper over raw inputs. - Preserve confusion matrix counts (true_positives, true_negatives, false_positives, false_negatives) as Python integers in serialized operating points performance estimates. - Correct PPCR docstring wording in create_probs_histogram() to 'Predicted Positives Condition Rate'. - Add focused tests verifying non-invocation of statistical producers by the low-level builder, spec parity between wrapper and low-level builder, integer confusion matrix types, and None/null serialization for undefined continuous metrics. Co-authored-by: uriahf <11351434+uriahf@users.noreply.github.com>
…tarball verification Update the wheel package verification assertion in .github/workflows/python-package.yml to check for rtichoke-viz-0.22.0.tar.gz instead of the superseded 0.20.2 tarball. Co-authored-by: uriahf <11351434+uriahf@users.noreply.github.com>
…er and vendor rtichoke_viz v0.22.0 - Vendor rtichoke_viz v0.22.0 release asset (SHA-256 64087fe0284ab2beb6e664dd419504a56000e217eaf403f33559896b67fff94d). - Implement two-layer PredictionDistributionSpec builder (_prediction_distribution_v2_spec and _prediction_distribution_v2_spec_from_performance_data) in _viz_spec_v2.py. - Export public create_probs_histogram() function in probs_distribution.py with boundary validation and Predicted Positives Condition Rate docstring terminology. - Update RtichokeBrowserChart to support single-file self-contained HTML rendering for prediction distribution charts. - Update .github/workflows/quarto-acceptance.yml and .github/workflows/python-package.yml for browser acceptance and wheel verification. - Add deterministic unit tests, authoritative JSON schema validation tests, negative contract tests, and Playwright real-browser acceptance tests. Co-authored-by: uriahf <11351434+uriahf@users.noreply.github.com>
…ogram-8470295566618691985-5908255235920094156 Update PR #417 to published rtichoke_viz v0.22.1 release artifact
|
Superseded by #418, which contains the same standalone Prediction Distribution implementation together with the final published rtichoke_viz v0.22.1 artifact. |
…er and vendor rtichoke_viz v0.22.0