[Repo Assist] fix: remove numpy/scipy symbol re-exports from datasets module (issue #981)

[github-actions]

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

Summary

Closes #981.

Issue #981 reports that dowhy.datasets.choice and dowhy.datasets.random appear in the generated documentation as if they were part of DoWhy's own API (they aren't — they leak in from from numpy.random import choice). The same problem affects bernoulli, halfnorm, poisson, and uniform from scipy.stats.

Root Cause

datasets.py had two bare star-like imports at module level:

from numpy.random import choice                             # leaked: choice
from scipy.stats import bernoulli, halfnorm, poisson, uniform  # leaked: 4 scipy objects

Because there is no __all__ in the module, every top-level name is exported. Sphinx picks them up and shows them in the module docs with broken [source] links (because their real source is in numpy/scipy, not datasets.py).

Fix

• Replace all three choice(...) call sites with np.random.choice(...) and remove the import.
• Replace all bernoulli/halfnorm/poisson/uniform call sites in sales_dataset() with ss.bernoulli/ss.halfnorm/ss.poisson/ss.uniform — scipy.stats is already aliased as ss — and remove the from scipy.stats import … line.

No functional change; behaviour is identical.

Test Status

• ✅ File parses cleanly (ast.parse)
• ✅ No remaining bare references to choice, bernoulli, halfnorm, poisson, or uniform (verified via grep)
• ℹ️ Full test suite could not be run in this environment (dependencies not installed), but the change is purely a rename of local symbols to their qualified forms — no logic is altered.

Note

🔒 Integrity filtering filtered 109 items

Integrity filtering activated and filtered the following items during workflow execution.
This happens when a tool call accesses a resource that does not meet the required integrity or secrecy level of the workflow.

• #1418 (list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• #1399 (list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• #1396 (list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• #1392 (list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• #1391 (list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• #1371 (list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:ImportError: weighting_sampler is not an existing do sampler. #71 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:DoSampler: Fails when you Add a Graph to the Causal Model #83 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:Errors of Notebook: The Causal Story Behind Hotel Booking Cancellations #198 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:Mediation Analysis key error #214 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:arrays used as indices must be of integer (or boolean) type #225 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:effect_estimate different using do operator and _estimate_effect function #309 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:AttributeError: 'CausalEstimate' object has no attribute '_estimator_object' #357 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:CausalModel.estimate_effect - UnboundLocalError: local variable 'identifier_name' referenced before assignment #365 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:Error in Conditional Effect Estimation in backdoor.linear_regression_estimator #401 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• issue:'fit_estimator' argument in 'estimate_effect' function of 'CausalModel' object #414 (list_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".)
• ... and 93 more items

Generated by Repo Assist · ◷

To install this agentic workflow, run

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

[emrekiciman]

lgtm, and passes tests.

[emrekiciman]

looks like this problem doesn't exist in the current docs. closing without merging.