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

New test subset script #548

Closed
wants to merge 7 commits into from
Closed

New test subset script #548

wants to merge 7 commits into from

Conversation

MattWellie
Copy link
Contributor

@MattWellie MattWellie commented Sep 15, 2023

A big ol' ton of changes

  • Changes all the queries from individual API queries to GraphQL queries. The number of queries could probably be shrunk down even further with some thought
  • Updates/Writes still use the fixed API endpoints/models
  • Arguments loosened up to allow any combination of #random samples, #random families, specific families and specific samples

I've got a fair bit of remaining uncertainty here - the original script had participants, samples, families, this one requires a different definition of Sample, and requires Sequencing Groups as an additional layer

@vivbak
Copy link
Contributor

vivbak commented Sep 18, 2023

Hey Matt, sorry does this overlap much with this PR? #519

@MattWellie
Copy link
Contributor Author

See #519

@MattWellie MattWellie closed this Sep 18, 2023
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