jtextor / dagitty

Graphical analysis of structural causal models / graphical causal models.
GNU General Public License v2.0
282 stars 46 forks source link

fix m-bias graph, for tests #18

Closed benibela closed 4 years ago

jtextor commented 6 years ago

Why is this necessary? It seems odd to revert this example back to the old syntax. Part of the point of this example was to show some features of the new syntax, such as the bounding box feature and the double-headed arrows.

For the unit tests, it is possible to add other graphs elsewhere rather than in this file, which is used by the GUI?

benibela commented 6 years ago

It is the last tetrad test case that fails.

Could change findExample to parse the new syntax, or put the graph in the test case with $p(...).

jtextor commented 4 years ago

I have now changed the findExample code to parse also the new syntax. All tests work now. Closing this issue.