Skip to content
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

Markov Check, record lower recall nodes when plotting data #1788

Merged
merged 3 commits into from
Jun 25, 2024

Conversation

vbcwonderland
Copy link
Collaborator

@vbcwonderland vbcwonderland commented Jun 25, 2024

Introduce a hard coded value of lowRecallBound for functions: getAndersonDarlingTestAcceptsRejectsNodesForAllNodesPlotData(fisherZTest, estimatedCpdag, trueGraph, 0.05, 1.0, 0.8)
and
getAndersonDarlingTestAcceptsRejectsNodesForAllNodesPlotData2(fisherZTest, estimatedCpdag, trueGraph, 0.05, 1.0, 0.8)
the examples above are setting the low recall bound as 0.8, which means all the low recalls strictly lower than this value would be recorded as a seperate list. The low record nodes are recorded in:
lowAdjRecallNodes.csv , lowAHRecallNodes.csv, lowLGRecallNodes.csv

@vbcwonderland vbcwonderland requested a review from jdramsey June 25, 2024 17:31
@vbcwonderland vbcwonderland changed the title record lower recall nodes Markov Check, record lower recall nodes when plotting data Jun 25, 2024
Copy link
Collaborator

@jdramsey jdramsey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I this I merged the PRs in the wrong order; you may want to double check my work.

@jdramsey jdramsey merged commit ad3edc4 into development Jun 25, 2024
@jdramsey jdramsey deleted the vbc-2024-06-20 branch June 25, 2024 18:11
vbcwonderland added a commit that referenced this pull request Jun 25, 2024
jdramsey added a commit that referenced this pull request Jun 25, 2024
Padding for PR #1788 Markov Check, record lower recall nodes when plotting data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants