Skip to content

[ENH] Add Lucas0 Dataset#9

Open
KahnSvaer wants to merge 3 commits intopgmpy:mainfrom
KahnSvaer:feature/add-lucas-dataset
Open

[ENH] Add Lucas0 Dataset#9
KahnSvaer wants to merge 3 commits intopgmpy:mainfrom
KahnSvaer:feature/add-lucas-dataset

Conversation

@KahnSvaer
Copy link
Contributor

  • Added Lucas0 Dataset from https://www.causality.inf.ethz.ch/data/LUCAS.html
  • Created DAG ground truth file based on accompanying picture of DAG in the source
  • Added snippet of DAG image for verification and record keeping
  • Added internal README.md
  • NOTE: LUCAS has not yet been added to the main project README to avoid merge conflicts; this can be done prior to final merge

@ankurankan @DARHWOLF Please review

Dataset addition for issue pgmpy/pgmpy#2790

Copy link

@DARHWOLF DARHWOLF left a comment

Choose a reason for hiding this comment

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

  • license file to be added
  • needs to be added to repository readme.

Copy link

Choose a reason for hiding this comment

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

Same comments as in #7

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