cmu-phil / tetrad

Repository for the Tetrad Project, www.phil.cmu.edu/tetrad.
GNU General Public License v2.0
406 stars 111 forks source link

Unable to Import Bootstrap Search Result Graph From File #1425

Closed kvb2univpitt closed 1 year ago

kvb2univpitt commented 1 year ago

A result graph can be exported after running search using bootstrapping, as shown here:

Screenshot from 2023-01-10 10-05-37

However, the exported graph could not be imported into a Graph Node.

Note that a result graph obtained from running search without bootstrapping can be exported and imported without any problem.

MikeKonrad commented 1 year ago

Kevin, I’ve had similar experience before. If I understand what you are describing correctly, what happens is something like this (from memory): when you do a bootstrapped search, the graph specification that is output includes an additional section not present in a graph specification from a regular search (not invoking boostrapping). That additional section is called something like “Graph Attributes” and it reports by node, the associated BIC score for that node. The Graph box does not know how to read/parse this information and so loading won’t work, but if you instead manually or programmatically intercept the loading of such a bootstrapped-search-result-graph-specification and strip away the “Graph Attributes” section it will load just fine.

From: Kevin Bui @.> Sent: Tuesday, January 10, 2023 10:15 AM To: cmu-phil/tetrad @.> Cc: Subscribed @.***> Subject: [cmu-phil/tetrad] Unable to Import Bootstrap Search Result Graph From File (Issue #1425)

A result graph can be exported after running search using bootstrapping, as shown here:

[Screenshot from 2023-01-10 10-05-37]https://user-images.githubusercontent.com/10729309/211587282-f5506338-b0b6-4029-bb75-365c7c3567d5.png

However, the exported graph could not be imported into a Graph Node.

Note that a result graph obtained from running search without bootstrapping can be exported and imported without any problem.

— Reply to this email directly, view it on GitHubhttps://github.com/cmu-phil/tetrad/issues/1425, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACI46PT7KJZIJYCOGGRILF3WRV4IPANCNFSM6AAAAAATW52YKE. You are receiving this because you are subscribed to this thread.Message ID: @.**@.>>

kvb2univpitt commented 1 year ago

@MikeKonrad Yes, you are correct! It seems like the parser doesn't know how to parse additional information from bootstrapping.

jdramsey commented 1 year ago

This has been fixed, all! :-D @kvb2univpitt I hope you don't mind my closing it. See release 7.1.3-1,