Update Annotation Configuration to include new annotations from Alliance Genome #81
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…t identifiers the models are now being pulled in as expected
|
|
|
Bonus verification: |
SeriousHorncat
approved these changes
May 16, 2023
SeriousHorncat
pushed a commit
that referenced
this pull request
Jun 6, 2023
…nce Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Fixed tests due to more annotations in the configuration, 4 to be exact * updated integration test annotation call values
SeriousHorncat
added a commit
that referenced
this pull request
Jun 6, 2023
* Update Annotation Configuration to include new annotations from Alliance Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Create Animal Model Card to represent the animal models from Alliance Genome (#86) * Adding scrolling to the cards * Changed the model annotations to get a limit of 100 models, rather than 20 models. Updated the fixtures to contain ALL models * Removed display: block, position: relative from card-base. Changed li margin-left to use variable. Removed the overflow-y from card-source * Consolidated duplicate conditional code to determine if a section is empty and change the text color in AlliangeGenomeCard * Updated the AllianceGenomeCard.spec.js toContain to be to.include * Merged and updated for the new configuration to allign with the new datasets being introduced. --------- Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Co-authored-by: James Scherer <jscherer@uab.edu>
JmScherer
added a commit
that referenced
this pull request
Jun 27, 2023
* Update Annotation Configuration to include new annotations from Alliance Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Create Animal Model Card to represent the animal models from Alliance Genome (#86) * Adding scrolling to the cards * Changed the model annotations to get a limit of 100 models, rather than 20 models. Updated the fixtures to contain ALL models * Removed display: block, position: relative from card-base. Changed li margin-left to use variable. Removed the overflow-y from card-source * Consolidated duplicate conditional code to determine if a section is empty and change the text color in AlliangeGenomeCard * Updated the AllianceGenomeCard.spec.js toContain to be to.include * Merged and updated for the new configuration to allign with the new datasets being introduced. --------- Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Co-authored-by: James Scherer <jscherer@uab.edu>
SeriousHorncat
added a commit
that referenced
this pull request
Jun 28, 2023
* Update Annotation Configuration to include new annotations from Alliance Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Create Animal Model Card to represent the animal models from Alliance Genome (#86) * Adding scrolling to the cards * Changed the model annotations to get a limit of 100 models, rather than 20 models. Updated the fixtures to contain ALL models * Removed display: block, position: relative from card-base. Changed li margin-left to use variable. Removed the overflow-y from card-source * Consolidated duplicate conditional code to determine if a section is empty and change the text color in AlliangeGenomeCard * Updated the AllianceGenomeCard.spec.js toContain to be to.include * Merged and updated for the new configuration to allign with the new datasets being introduced. --------- Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Co-authored-by: James Scherer <jscherer@uab.edu>
SeriousHorncat
added a commit
that referenced
this pull request
Jul 5, 2023
* Update Annotation Configuration to include new annotations from Alliance Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Create Animal Model Card to represent the animal models from Alliance Genome (#86) * Adding scrolling to the cards * Changed the model annotations to get a limit of 100 models, rather than 20 models. Updated the fixtures to contain ALL models * Removed display: block, position: relative from card-base. Changed li margin-left to use variable. Removed the overflow-y from card-source * Consolidated duplicate conditional code to determine if a section is empty and change the text color in AlliangeGenomeCard * Updated the AllianceGenomeCard.spec.js toContain to be to.include * Merged and updated for the new configuration to allign with the new datasets being introduced. --------- Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Co-authored-by: James Scherer <jscherer@uab.edu>
SeriousHorncat
added a commit
that referenced
this pull request
Jul 7, 2023
* Update Annotation Configuration to include new annotations from Alliance Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Create Animal Model Card to represent the animal models from Alliance Genome (#86) * Adding scrolling to the cards * Changed the model annotations to get a limit of 100 models, rather than 20 models. Updated the fixtures to contain ALL models * Removed display: block, position: relative from card-base. Changed li margin-left to use variable. Removed the overflow-y from card-source * Consolidated duplicate conditional code to determine if a section is empty and change the text color in AlliangeGenomeCard * Updated the AllianceGenomeCard.spec.js toContain to be to.include * Merged and updated for the new configuration to allign with the new datasets being introduced. --------- Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Co-authored-by: James Scherer <jscherer@uab.edu>
SeriousHorncat
added a commit
that referenced
this pull request
Jul 7, 2023
* annotation rendering configuration frontend updates (#84) * updated annotation configuration, updated unit tests, partially updated system tests * updating the sidebar style to remained a fixed size, updated the logic for removing '_' from the anchors when displaying them * fixed system test & made the anchors have consistent naming style * added back the dataset data for each model * lint fix * unit test update * removed file upload where not needed * made requested PR changes, updating variant to hgvs_variant for attachment/genomic Type type, update styling to use variable set in main.css * Update frontend/src/models/analyses.js Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Signed-off-by: Alex Moss <kuthedk@users.noreply.github.com> * Update frontend/src/models/analyses.js Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Signed-off-by: Alex Moss <kuthedk@users.noreply.github.com> * Update frontend/src/models/analyses.js Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Signed-off-by: Alex Moss <kuthedk@users.noreply.github.com> * Update case_annotation_section_anchors.cy.js * Update frontend/src/models/analyses.js Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Signed-off-by: Alex Moss <kuthedk@users.noreply.github.com> --------- Signed-off-by: Alex Moss <kuthedk@users.noreply.github.com> Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> * Alliance genome models into new configuration (#88) * Update Annotation Configuration to include new annotations from Alliance Genome (#81) * Now fetching the proper alliance genomic identifiers, with the correct identifiers the models are now being pulled in as expected * Create Animal Model Card to represent the animal models from Alliance Genome (#86) * Adding scrolling to the cards * Changed the model annotations to get a limit of 100 models, rather than 20 models. Updated the fixtures to contain ALL models * Removed display: block, position: relative from card-base. Changed li margin-left to use variable. Removed the overflow-y from card-source * Consolidated duplicate conditional code to determine if a section is empty and change the text color in AlliangeGenomeCard * Updated the AllianceGenomeCard.spec.js toContain to be to.include * Merged and updated for the new configuration to allign with the new datasets being introduced. --------- Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Co-authored-by: James Scherer <jscherer@uab.edu> * Migration script to update the data layer for the additional sections (#89) * Created migration script file to add additional annotation sections * Migration script saving changes in database. Debugging comments removed * Update etc/fixtures/migrations/migrate-additional-sections.js Added suggestion to update section name to match configuration Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Signed-off-by: Rabab Fatima <fr.rabab@gmail.com> --------- Signed-off-by: Rabab Fatima <fr.rabab@gmail.com> Co-authored-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> * Investigate why CPAM0084 breaks when re-running annotation from the api docs (#91) * Updated the fixtures for CPAM0084 * Updated python tests * As a Computational Biologist, I want to see the various summaries for animal models in the relevant section so I can have all the information consolidated in one location (#92) * Updated annotation configuration to have both automated and specificed alliance genome animal summaries, updated the fixtures to include both summaries and are accurate to the gene identifier, and updated the annotation rendering configuration to display the new annotations * Updated the python tests to include the 4 extra annotations in the configuration * updated the existing tests and removed the mongo ids from the imported fixtures * updated the existing tests and removed the mongo ids from the imported fixtures * Updated the Text Dataset in the Annotation components to display as unavailable if the value also has empty string * Updated the fixtures for genomic unit collection * Removed the mongoids from the genomic units collection initial seed * additional annottations * resolved unit test failures * Fixing the backend tests * Updated the integration tests for backend * As a Developer, provide a way to remove all automatic annotations from analyses, so they can be re-annotated with the correct information (#99) * Adding the remove annotation and annotation queueing scripts * Adding documentation and run feedback * Updated the analysis annotation script to give relevant output * Fixed the annotation config that broke annotation * Linting should be passing * Removed debugging print statements * Added a note to the annotate-all-existing-analyses.sh that it may need to be run more than once * Removed a space * Updated the the annotation configuration to uitilize the tag dataset to visualize the HPO terms * Fixed the unit test failing due to the addition of the tag datasets component * fixed the mised merge issue * pushing up system test changes * Skipping the 'should allow the user to navigate the analysis via the logo, header, and section anchor links' for now until we fix it --------- Signed-off-by: Alex Moss <kuthedk@users.noreply.github.com> Signed-off-by: Angelina Uno-Antonison <ange.unoantonison@gmail.com> Signed-off-by: Rabab Fatima <fr.rabab@gmail.com> Co-authored-by: Alex Moss <kuthedk@users.noreply.github.com> Co-authored-by: James Scherer <jscherer@uab.edu> Co-authored-by: Rabab Fatima <rfatima@uab.edu>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the annotation data part of updating the animal models from Alliance Genome.
Pull Request Details
https://www.wrike.com/open.htm?id=1112499160
https://www.wrike.com/open.htm?id=1112498242
Changes made:
To Review:
# From the root of Rosalution docker-compose down docker system prune -a --volumes docker-compose up --build -ddeveloperCreate New Analysiscardetc/fixtures/C-PAM0074.sondocker exec -it rosalution-rosalution-db-1 mongosh rosalution_dbGRIN2Dgene in the genomic_units collectiondb.genomic_units.find({"gene":"GRIN2D"})Mongo Collection Entry:
Alliance Genome Table:
Alliance Genome API:
Bonus - Check the Rosalution API for GRIN2D to see if the data is there: