Fix for bugs with displaying sequences #1136
Merged
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.
Bug Fix
Issue #1135
Description
This PR fixes two problems
so__cdsanddata__sequence_coordinatesfields will cause an mRNA page to not load if the reference genome sequences are loaded and memory is set too low (but not unreasonable). This is caused by a call tochado_expand_varthat includes thefeature.residuescolumn of the reference sequence.feature.residuesfield and will not show derived sequences from the reference. Most notably, the mRNA pages were missing the full length mRNA (with introns spliced out), and parent gene sequences.Testing?
Note: The only way to test the first bug is to load a large genome (~3GB) with large supercontigs. If you are not able to do this or don't have the time, then I think just checking existing gene and mRNA pages to make sure this code doesn't break them is sufficient. If you don't have mRNA/gene pages then loading of the Citrus demo data from the Tripal tutorial should work.
drush updatedbto push a new database update. It will tell you a new sequence field is added.feature. residuescolumn as it pulls them from the reference sequence./web-services/content/v0.1/mRNA