Skip to content

[Repo Assist] improve(gcm): export missing public symbols from dowhy.gcm - #1724

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
repo-assist/improve-gcm-missing-exports-20260731-50891101d51cfca9
Draft

[Repo Assist] improve(gcm): export missing public symbols from dowhy.gcm#1724
github-actions[bot] wants to merge 1 commit into
mainfrom
repo-assist/improve-gcm-missing-exports-20260731-50891101d51cfca9

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

🤖 This is an automated PR from Repo Assist, an AI assistant for this repository.

Summary

Four public GCM symbols have comprehensive docstrings and are tested, but are not accessible via import dowhy.gcm — forcing users to reach into internal sub-modules.

Symbol Sub-module Issue
RankBasedAnomalyScorer dowhy.gcm.anomaly_scorers Referenced by name in the MedianCDFQuantileScorer docstring as "a statistically more rigorous alternative" — users expect to find it next to the other scorers
conditional_anomaly_scores dowhy.gcm.anomaly Direct building block of anomaly_scores(); used in tests via from dowhy.gcm.anomaly import ...
arrow_strength_of_model dowhy.gcm.influence Per-mechanism companion to arrow_strength(); appears in the confidence_intervals.py docstring example (direct_arrow_strength_of_model) and is tested
intrinsic_causal_influence_sample dowhy.gcm.influence Sample-level companion to intrinsic_causal_influence(), fully documented with a research paper reference and tested

Change

Single file changed — dowhy/gcm/__init__.py:

  • Added conditional_anomaly_scores to the .anomaly import line
  • Added RankBasedAnomalyScorer to the .anomaly_scorers block
  • Added arrow_strength_of_model and intrinsic_causal_influence_sample to the .influence block

No logic changes anywhere.

Test Status

32 passed in 161s

Tests run: test_anomaly_scorers.py, test_anomaly.py, test_arrow_strength.py, test_intrinsic_influence.py

Format: ✅ poe format_check passes
Lint: ✅ poe lint passes (no new issues)

Generated by 🌈 Repo Assist, see workflow run. Learn more.

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@11c9a2c442e519ff2b427bf58679f5a525353f76

Four symbols are documented, tested, and used by name in docstrings or
user-facing docs but were absent from dowhy/gcm/__init__.py, forcing
users to reach into sub-modules directly:

* RankBasedAnomalyScorer      – mentioned by name in MedianCDFQuantileScorer
                                 docstring as a 'more rigorous' alternative;
                                 tests already import from dowhy.gcm.anomaly_scorers
* conditional_anomaly_scores  – a useful building block of anomaly_scores();
                                 tests already import from dowhy.gcm.anomaly
* arrow_strength_of_model     – per-mechanism companion to arrow_strength();
                                 used in confidence_intervals.py docstring example
                                 and in tests
* intrinsic_causal_influence_sample – sample-level companion to
                                 intrinsic_causal_influence(); fully documented
                                 and tested, but not reachable from dowhy.gcm

All four symbols are now exported via dowhy/gcm/__init__.py.
No logic changes; tests unchanged (all 32 pass).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants