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

Try to merge VBN 2022 and VCN 2019 LIMS APIs #2411

Open
5 tasks
danielsf opened this issue May 5, 2022 · 0 comments
Open
5 tasks

Try to merge VBN 2022 and VCN 2019 LIMS APIs #2411

danielsf opened this issue May 5, 2022 · 0 comments

Comments

@danielsf
Copy link
Contributor

danielsf commented May 5, 2022

#2407 created a lims_queries.py module to handle all of the LIMS database queries necessary to generate the metadata tables for the VBN 2022 release. There is an ecephys-related LIMS API here

https://github.com/AllenInstitute/AllenSDK/blob/master/allensdk/brain_observatory/ecephys/ecephys_project_api/ecephys_project_lims_api.py

however, it makes several strong assumptions about the where clauses that our stakeholders did not want to make when generating the VBN data release, specifically, the selection on WHERE ecephys_probes.valid_data. See

#2407 (comment)

for a brief discussion.

The purpose of this ticket is to try to modify the old LIMS API so that the lims_queries.py module created for the VBN 2022 metadata writer can rely on that code

Tasks

  • Generate baseline metadata tables using python -m allensdk.brain_observatory.vbn_2022.metadata_writer. Save for validation.
  • Add ability to old ecephys LIMS API to specify probe_ids_to_skip
  • Add ability told ecephys LIMS API to turn off the hard selection on WHERE ecephys_probes.valid_data
  • Modify VBN 2022 metadata writer lims_queries.py to use the old ecephys LIMS API to do its queries.

Validation

  • Re-generate the metadata tables generated in the first step above. Make sure that their contents have not changed (or, if they have, make sure that there is a sound reason for that change).
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

No branches or pull requests

1 participant