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
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
|
SeriousHorncat
approved these changes
Jun 14, 2023
…d 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
…navailable if the value also has empty string
JmScherer
force-pushed
the
animal-model-summary-updates
branch
from
6878da0
to
8eb07e2
Compare
JmScherer
merged commit 5b17244
into
annotation-rendering-configuration-new-section-updates
JmScherer
added a commit
that referenced
this pull request
Jun 27, 2023
… 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
SeriousHorncat
pushed a commit
that referenced
this pull request
Jun 28, 2023
… 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
SeriousHorncat
pushed a commit
that referenced
this pull request
Jul 5, 2023
… 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
SeriousHorncat
pushed a commit
that referenced
this pull request
Jul 7, 2023
… 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
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.
Checklist before requesting a review
Pull Request Details
This pull request adds new annotations for the animal model summaries available from Alliance Genome (AG). The new summaries are the AG Automated Summary and the Animal Database Summary (MGI, ZFIN, WB, etc).
https://www.wrike.com/open.htm?id=1140797927
Changes made:
Mouse_Alliance_Genome_MGI_SummaryRat_Alliance_Genome_RGD_SummaryZebrafish_Alliance_Genome_ZFIN_SummaryC-Elegens_Alliance_Genome_WB_SummaryModel Systems - Rat->Rat_Alliance_Genome_Automated_SummaryModel Systems - Mouse - Automated->Mouse_Alliance_Genome_Automated_SummaryModel Systems - Zebrafish - Automated->Zebrafish_Alliance_Genome_Automated_SummaryModel Systems - C-Elegens - Automated->C-Elegens_Alliance_Genome_Automated_SummaryTo Review:
# From root of Rosalution docker compose down docker system prune -a --volumes docker-compose up --builddeveloperetc/fixtures/import/C-PAM0074.jsonExisting Fixtures: CPAM0002
Uploaded Analysis: CPAM0074