CausalDiscoveryToolbox icon indicating copy to clipboard operation
CausalDiscoveryToolbox copied to clipboard
verified publisher icon FenTechSolutions

GIES targets and target.index parameter needs to be exposed

Open shahashka opened this issue 3 years ago • 2 comments

The cdt.causality.graph.GIES constructor is missing parameters to set the targets and target.index parameters for the score functions GaussL0penIntScore and GaussL0penObsScore. These parameters allow the user to specify the interventional targets and which data row has which intervention applied -- this is needed for the GIES algorithm. See https://search.r-project.org/CRAN/refmans/pcalg/html/GaussL0penIntScore-class.html

As is, the gies.R code uses source$getTargets() which by default sets targets to observational targets only. This means right now, the GIES algorithm in cdt does not handle interventional data properly.

shahashka avatar Sep 19 '22 16:09 shahashka

Hello ! Yes I forgot about this ! It's in the roadmap and I need to update this.

Thank you, noted. Best, Diviyan

diviyank avatar Sep 20 '22 09:09 diviyank

Ok cool! Patiently will wait for this update :) (Do you have an approximate timeline?) Thanks for the reply

shahashka avatar Sep 20 '22 16:09 shahashka