tetrad icon indicating copy to clipboard operation
tetrad copied to clipboard

BPC is missing a critical check

Open cg09 opened this issue 5 years ago • 5 comments

Both BPC and FOFC require a preprocessing step that checks to see whether the marginal graphical model on the measured variables is complete--or nearly so--and only runs the latent variable search of BPC or FOFC if it is complete. I have asked Madelyn to look into this problem, but we may have to hand it off.

I fixed this for FOFC--jdramsey

cg09 avatar Jun 16 '19 17:06 cg09

@cg09 @mglymour Was this fixed? If so, in what branch? Thanks.

jdramsey avatar Aug 17 '19 21:08 jdramsey

@cg09 @mglymour Asking again, was this fixed? I know it was fixed in Madelyn's code, but was that ever pushed to development?

jdramsey avatar Feb 20 '20 15:02 jdramsey

Only Madelyn would know.

On Thu, Feb 20, 2020 at 10:33 AM Joseph Ramsey [email protected] wrote:

@cg09 https://github.com/cg09 @mglymour https://github.com/mglymour Asking again, was this fixed? I know it was fixed in Madelyn's code, but was that ever pushed to development?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cmu-phil/tetrad/issues/1162?email_source=notifications&email_token=AD4Y3OIMDD6ISRQAOSVLAS3RD2PD7A5CNFSM4HYRV46KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMOWYFA#issuecomment-589130772, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD4Y3OKWK3RKSLKQZCA7FJ3RD2PD7ANCNFSM4HYRV46A .

cg09 avatar Feb 20 '20 16:02 cg09

No, we have not implemented a satisfactory check, although we have sketched a procedure.

On Thu, Feb 20, 2020 at 10:33 AM Joseph Ramsey [email protected] wrote:

@cg09 https://github.com/cg09 @mglymour https://github.com/mglymour Asking again, was this fixed? I know it was fixed in Madelyn's code, but was that ever pushed to development?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cmu-phil/tetrad/issues/1162?email_source=notifications&email_token=AD4Y3OIMDD6ISRQAOSVLAS3RD2PD7A5CNFSM4HYRV46KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMOWYFA#issuecomment-589130772, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD4Y3OKWK3RKSLKQZCA7FJ3RD2PD7ANCNFSM4HYRV46A .

cg09 avatar Feb 20 '20 16:02 cg09

OK. I shall wait for truth to be divulged.

jdramsey avatar Feb 20 '20 16:02 jdramsey