-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ability to generate causal graphs #68
Comments
Hi @hannah-rae, @urebbapr , @wkiri , @emhuff , I've checked the initial implementation of causal graphs in the Example outputs of causal graphsThe example outputs of causal graphs generated using
Implementation summaryCausal graphs are currently implemented together with the There is one issue that I don't know how to resolve yet. Causal graphs are generated using classes/functions in fges-py github repository, but this repository isn't installable (the authors don't provide a setup.py script). This isn't a big problem for us to use causal graphs on UMD/JPL machines. We can manually Use causal graphsFor now, causal graphs must be generated with
There will be one causal graph generated per cluster group, and the causal graphs will be saved in the directory defined by |
Please note that I am aware of the build failures (code formatting issues, please see the screenshot below) caused by the implementation of the causal graph. I can't fix these code formatting issues because I have to use |
Below is a temporary solution to install DORA with causal graphs (for @hannah-rae to install it on UMD machine).
dora/dora_exp_pipeline/dora_results_organization.py Lines 8 to 11 in 0b8b675
|
@stevenlujpl I think these look great. If you have time for tiny updates, I suggest |
@wkiri Thanks for the comments. I've incorporated them into the code. In addition, I also added the |
@stevenlujpl The updated visualization looks fantastic! |
@stevenlujpl Is this ready to be closed now? |
@hannah-rae, Not yet. Currently, all the updates for causal graphs are in
|
No description provided.
The text was updated successfully, but these errors were encountered: