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

add genotype_call_results_reporter script #514

Merged
merged 4 commits into from
May 24, 2018
Merged

Conversation

ces
Copy link
Contributor

@ces ces commented May 17, 2018

  • Note when env variable dev is set the LIMS dev url returned by st::api::base is on port 6610 which is the training url. Currently only the default LIMs dev url on port 6600 supports the required api endpoint (which can be given via the api_url command line option).
  • Multiple results per sample uuid can be posted to the LIMs. This takes account of the fact that a sample could be run on different primer panels or re-run with the same panel on a different run. However it also means if a duplicate result were to be reported it would also be entered into the LIMs successfully but with a more recent date. We assume the most recent result for a particular sample and panel combination would be used in downstream reporting.

@mgcam mgcam self-assigned this May 23, 2018
@mgcam mgcam merged commit 348af66 into wtsi-npg:devel May 24, 2018
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.

4 participants