Skip to content

feat!: use materialized view for tx-exon alignment lookup#462

Merged
jsstevenson merged 1 commit intomainfrom
refactor/461-use-mv
Mar 13, 2026
Merged

feat!: use materialized view for tx-exon alignment lookup#462
jsstevenson merged 1 commit intomainfrom
refactor/461-use-mv

Conversation

@jsstevenson
Copy link
Member

close #461

  • Use the equivalent materialized view for this lookup. It's a small speedup but it only took a few minutes to fix (I think the view just exists as a referent for building the materialized view)
  • Rename methods regarding "tx_exon_aln_v" to just "tx_exon_aln", since the Python user doesn't really care that it's a view or a materialized view or whatever, they just need the "tx-exon aln" part. Hopefully this isn't too annoying to change wherever else it's used but if it is, this part could be walked back

@jsstevenson jsstevenson requested a review from a team as a code owner March 13, 2026 19:56
@jsstevenson jsstevenson added the priority:low Low priority label Mar 13, 2026
@jsstevenson jsstevenson merged commit 8a11e5e into main Mar 13, 2026
15 checks passed
@jsstevenson jsstevenson deleted the refactor/461-use-mv branch March 13, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:low Low priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use tx_exon_aln_mv, not tx_exon_aln_v

2 participants