Transcriptomic POC - #49
Draft
TamaraNaboulsi wants to merge 5 commits into
Draft
TamaraNaboulsi wants to merge 5 commits into
TamaraNaboulsi wants to merge 5 commits into
Conversation
TamaraNaboulsi
requested review from
darefalola,
dpopleton,
sgiorgetti and
veidenberg
September 29, 2026 13:32
Contributor
|
In general looks good. Few small suggestions:
|
This file contains hidden or 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
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 (very basic) initial change to handle transcriptomic tracks. The changes here are by no means final and are open to changing or scraping altogether based on better understanding of the project and/or better implementation ideas. This is how I understood the starting point of the backend side and it is limited to track discovery at this point. It includes the addition of a new model to represent the transcriptomic configuration, an update to the endpoint
track_categoriesto return a new category of tracks (under the parent category of Genomic), a new endpoint which returns the configuration itself, and some seeding/preparation code that adds the POC pig data into an existing sqlite db.The flow I envisioned:
track_categoriesis called and returns a new section representing the transcriptomic tracks. In contrary to the other (called 'inline' in this POC) tracks, I did not see a need to return all the tracks at this point as the UI doesn't need to list them on the right side panel like the others. The idea is that for users who aren't interested in transcriptomic, this saves a bit of loading.transcriptomic/[genome-uuid]/configurationis called and that basically returns the json data. This returns everything for now, with the possibility of later choosing what we want returned at this point, especially for genomes with huge amounts of tracks.Notes & caveats:
Next steps:
Testing:
I have tested this code on the latest (release 30) tracks.sqlite3 db provided by Automation, applying the seeding script on it to populate it with the pig data. On a local implementation of the endpoint, these are currently the returned values.