-
Notifications
You must be signed in to change notification settings - Fork 85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rendering issue with neat features #2063
Comments
Just in case, the gff file I loaded in the merlin_html_2.gff track:
|
@abretaud Can you do a couple of things:
Its possible we aren't properly annotating CDS's, but I thought we would turn them into Exon's first. |
In case it helps, here's the html of the first gene in uca:
|
The exported cds fasta:
|
Okay.
https://github.com/abretaud/docker-apollo/blob/bipaa/apollo-config.groovy#L74-L90
|
|
I would remove the The default is false. Basically, this tries to use the existing CDS to calculate the new one. By default Apollo always tries to recalculate the most likely CDS based on the largest ORF. The most common use-case is to promote of bunch of existing predicted annotations, preserving their annotations. If you don't have a good reason for making that true, I wouldn't set it to true. |
It's the same when removing |
By the "same" did you mean, when you created an annotation again after removing the line (or setting it to false) and redeploying it? If its not too much time, you might want to try explicitly setting it to false, redeploying, and re-create the annotations. I'll try to take a closer look at it this week or early next. |
Yep, I meant there is still the problem after unsetting the env var (or setting it to false explicitly), redeploying, add a new organism with my jbrowse instance, and add genes to uca |
Okay .. thanks for testing all of these. I’ll take a look. If the jbrowse data directory is small enough (<5 gb) if you send me that I could probably make short work of this issue.
Nathan
… On Feb 21, 2019, at 8:30 AM, Anthony Bretaudeau ***@***.***> wrote:
Yep, I meant there is still the problem after unsetting the env var (or setting it to false explicitly), redeploying, add a new organism with my jbrowse instance, and add genes to uca
Thanks for the help!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I've just sent 2 sample data dirs by email, you received it? (sent to lbl address) |
I have not seen it yet. If I don’t get it by tomorrow, maybe we can chat on Gitter.
Nathan
… On Feb 21, 2019, at 8:44 AM, Anthony Bretaudeau ***@***.***> wrote:
I've just sent 2 sample data dirs by email, you received it? (sent to lbl address)
I have to leave now sorry
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#2063 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAt2qpXrQFY9KGgdkLGDhtFGmkxDmw89ks5vPszegaJpZM4bFaKV>.
|
Interesting . . .the GFF3 is correct, but for some reason the returned element has two exons (or an exon and a CDS) in separate places: The problem is that the exon has 3 children that are exons (?!?), two of which are the original correct exons.
Viewing it in the JSON viewer, it like there is an intermediate |
With the other one, its slightly different, but still not properly defined evidence: Basically, it should always go, mRNA -> (exon|CDS) and exons should not generally have subfeatures, though if there is a good argument, I can take a look. If |
Though, the evidence suggests that what you have is correct when I view details (either the CDS or exon version should have worked), so I'm unsure why we are getting two layers. Looking at the evidence, the GFF3 is a bit funky:
Fixed it here:
|
Actually, I think the |
I redid this with just a straight GFF3 with 2.3.1 and the same problem:
Testing with a 2.2.0 regression |
Same result for 2.2.0 . . . I'm wondering it he issue might be more related to it using the GFF3Tabix store versus the NCList one. This would be good to fix, as I would prefer the native stores. |
@abretaud I remember the problem. The issue is that when using the GFF3Tabix, it flips out if it has a top-level gene class. I changed it to be top-level mRNA:
and it works: |
This obviously isn't acceptable as GFF3's should have genes in them. Two solutions:
Anyway, this is critical for 2.4.0 |
Cool, thanks for looking into it! I won't be able to do it until tomorrow, but no problem if you want me to test some patch |
Nothing to do today (or tonight) for you. Hopefully I'll have something more working tomorrow so I can finish #2064 |
Testing Apollo 2.3.1 with NeathHTMLFeatures and NeatCanvasFeatures, I get this rendering issue:
Here's the track config for UCA:
And here's the config for the track I dragged models from (I left them untouched in the uca):
(I think I've seen it in another issue, but I can't find it... Sorry if it's a duplicate)
The text was updated successfully, but these errors were encountered: