-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Description
According to the documentation, the parameter domain_data in the function transport_conditional_counterfactual_query does not contain the data from the target domain:
y0/src/y0/algorithm/counterfactual_transport/api.py
Lines 2344 to 2345 in b1381a4
| :param domain_data: Corresponding to $\mathcal{Z}$ in [correa22a]_, this is a set of | |
| $K$ tuples, one for each of the $K$ domains except for the target domain. |
However, in the function, conditional_cft, the parameter domains does contain the target domain
y0/src/y0/algorithm/counterfactual_transport/api.py
Lines 2298 to 2299 in b1381a4
| :param domains: A set of $K$ CFTDomain classes, one for each of the $K$ domains. See the | |
| documentation for CFTDomain for more details. |
, and the target domain is passed as part of domain_data
y0/src/y0/algorithm/counterfactual_transport/api.py
Lines 2303 to 2313 in b1381a4
| domain_graphs = [ | |
| (domain.graph, domain.ordering or domain.graph.topological_sort()) for domain in domains | |
| ] | |
| domain_data = [(domain.policy_variables, domain.population) for domain in domains] | |
| return transport_conditional_counterfactual_query( | |
| outcomes=_event_from_counterfactuals_strict(outcomes), | |
| conditions=_event_from_counterfactuals_strict(conditions), | |
| target_domain_graph=target_domain_graph, | |
| domain_graphs=domain_graphs, | |
| domain_data=domain_data, | |
| ) |
Metadata
Metadata
Assignees
Labels
No labels