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

Pl/remove distribution table #4

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

paulflang
Copy link
Owner

This introduces a couple of changes disussed in the last meeting:

  • split up parameter table in a parameter table and random effects table
  • Allow multiple random effects per expression
  • rename the groupings (I used distributions now)

Additionally, I removed the individuals table. The information there is now implied in the measurements table, via the distributionId columns. These column can be used to express which datapoint belongs to which patient. Or if there are multiple groupings, like patient, tumorStage and eye color, then they define which patient has which tumorStage and which eye color at which time point. One and the same patient has to share the same covariates drawn from the patientId distribution. Ditto for tumorStage and eye color.

I also removed the distributionType column from the parameter table and put it into the YAML file.

@paulflang
Copy link
Owner Author

@eraimundez , what do you think in general?
And what do you think about having the random effect table listed at the same level of hierarchy as the parameter table in the YAML file?

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.

1 participant