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

Dev #4431

Merged
merged 93 commits into from
Oct 15, 2024
Merged

Dev #4431

merged 93 commits into from
Oct 15, 2024

Conversation

hanars
Copy link
Collaborator

@hanars hanars commented Oct 15, 2024

  • All unit test are passing and coverage has not substantively dropped
  • No new dependency vulnerabilities have been introduced
  • Any changes to the docker image have been vetted for potential vulnerabilities
  • If any python requirements are updated, they are noted and here and will be updated before deploying:
  • Any database migrations are noted here, and will be run before deploying:
    • Fix biosample options
  • All major changes are recorded in the changelog, and the changelog has been updated to reflect the latest release
  • Any new endpoints are explicitly tested to ensure they are only accessible to correctly permissioned users
  • No secrets have been committed
  • Any changes to environment variables or secrets are noted here, and will properly configured before deploying:
    • Add HAIL_SEARCH_DATA_DIR, unset LUGUI env vars in tgg-helm

hanars and others added 30 commits September 24, 2024 18:02
…rtup

allow hail backend to run without all data types
Genotype filters per sample instead of per variant
…l-docs

add auth to local install documentation
…OSTNAME` env vars. (#4405)

* env var consistency

* Change the port back

* more explicit

* Flip the ports
hanars and others added 29 commits October 10, 2024 17:07
* variant lookup table has sample_type

* project_sample_types

* with definitions

* try hail

* switch sample_type and family_guid in project_samples dict"
"

* more

* existing multi project search test passes

* lookup tests work

* make sure existing search tests work

* remove a print

* migration and lookup response

* js test fixtures

* first pass at handling edge cases is just join filtered out variants back

* more comments

* space

* add another family for unit testing

* reduce joins

* filter conditionally

* fix some bugs

* comments

* test fixture

* add another test case

* working on inheritance filter changes

* handle one edge case

* mixin class

* test project path

* fix seqr tests

* refactoring

* fix the diff

* keep _filter_families_inheritance

* undo some things

* remove constant

* move the merging

* n_partitions and genotype annotation

* undo es utils test changes"
git push

* fix variant search api tests

* lil fix

* merging still not working as expected

* apply_filters

* fix a few tests

* is_defined

* Array.isArray

* a few minor things

* a few changes

* rename

* make loading logic similar

* shared filter logic

* use hl.dict

* mirror base class

* recessive test case

* filtering

* enum!

* add test case

* extra fixtures

* data strcture for loading project hts and naming changes"

* cleanup

* refactor a bit

* even simpler

* tiny changes

* even simpler

* kwargs

* fix a bug

* reduce a wee bit of shrared code

* try to make diff better

* filters

* move fixtures

* coverage...

* some comments

* comment

* fix some fixtures

* comments

* generalize loading project hts and checking for multiple sample types on project table load"

* removed unused code and minor change

* sorted?

* bug fix

* cleaner

* search

* fixtures

* comments

* ui fixtures

* bug

* simplify kwargs

* starmap
Add editable fields to bulk update download
…-subset

Subset loading samples from airtable for SVs and MITO
Use boolean for sending sex metadata to hail search
@hanars hanars merged commit cc21f6e into master Oct 15, 2024
12 checks passed
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.

3 participants