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

Get AIP running on multiple datasets #490

Merged
merged 5 commits into from
Oct 24, 2023

Conversation

MattWellie
Copy link
Contributor

@MattWellie MattWellie commented Oct 24, 2023

Issue: Closes #489

  • AIP implementation here ran on the default dataset config[workflow][dataset]
  • When running on multiple cohorts, the dataset is always seqr
  • Alter scripts to take a dataset argument, which is used to read the specific parts of config.
  • Additional: Config was being sourced before AIP-specific default content was loaded, so the early get_config needed to be cut out

@MattWellie MattWellie merged commit 98c34e1 into main Oct 24, 2023
3 checks passed
@MattWellie MattWellie deleted the get_aip_running_multiple_datasets branch October 24, 2023 23:02
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.

AIP config is sourced too soon
2 participants