Conversation
|
|
||
| @property | ||
| def length(self) -> int: | ||
| """The total length of this transcript, summing all exon lengths (includes UTRs).""" |
There was a problem hiding this comment.
introns are part of a transcript... are we conflating this with transcriptome mrna?
There was a problem hiding this comment.
@gkaur can you comment on this?
The original request:
Currently, transcript length is calculated by iterating over the exons, computing the difference between each exon’s start and end positions, and summing those values. It would be much easier if this was recalculated and stored for each transcript.
There was a problem hiding this comment.
I was hoping to capture post processed transcript length based on exons and cdss components only
| return sum(len(exon) for exon in self.exons) | ||
|
|
||
| @property | ||
| def length_cds(self) -> int: |
There was a problem hiding this comment.
I feel like the naming will be confusing in code.
eg.
len(trans.cdss)
vs
trans.length_cds
There was a problem hiding this comment.
That's a reasonable concern. I'm coming up empty trying to think of a clearer alternative.
There was a problem hiding this comment.
How about .exonic_length and .coding_length?
No description provided.