Closed adityaekawade closed 6 years ago
Actually just found a bug. If you specify the .tbi file then two sampling events are fired. If you click the link below you can see the read sampled number go higher then lower then higher multiple times indicating two sampling commands have been executed.
I did some testing this morning. Overall, it is working well. Here are a few minor issues:
@tonydisera : Regarding (1), the filter sample button would appear only if the URL contains the samples. If it does not contain the samples, it would show just the load button. I am attaching the screenshots of both the cases.
Aditya, I am referring to the use case when the VCF URL is specified in the launch url. When vcf.iobio is launched, the 'Filter samples' doesn't appear even though the vcf url is a multi-sample vcf: http://localhost/vcf.iobio.io/?vcf=https%3A%2F%2Fs3.amazonaws.com%2Fiobio%2Fsamples%2Fvcf%2Fplatinum-exome.vcf.gz&tbi=&samples=NA12878&build=GRCh37
The behavior should be the same as when the user enters the vcf url on the landing page.
Ok. Got it! I noticed the same thing now even when the page is refreshed. Thank you.
I have fixed these issues in the latest commit:
@tonydisera @chmille4
Hi Aditya -
It looks like this fix may have broken something. When I enter a multi-sample vcf (from the landing page), the sample select dropdown no longer appears.
After clicking on load, the 'Filter samples' button does appear on the main page. And after launching vcf.iobio with the VCF URL now shows the 'Filter samples' button.
Hi Tony, Can you please confirm again, it is working fine for me.
Oops! You are right. It is fixed. Looks good. You can merge to gh-pages (and master) to deploy to production.
I just accepted this pull request and merged to gh-pages. Aditya, can you please confirm that everything is working on production? Thanks!
Yes, I am checking that! Thank you.
On 29 January 2018 at 10:57, Tony Di Sera notifications@github.com wrote:
I just accepted this pull request and merged to gh-pages. Aditya, can you please confirm that everything is working on production? Thanks!
— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/tonydisera/vcf.iobio.io/pull/59#issuecomment-361330348, or mute the thread https://github.com/notifications/unsubscribe-auth/APh8KSUwR74bQc2hRQU71Xk4IcWoaCE0ks5tPgaBgaJpZM4Rr1Zw .
It's working fine on production!
Lot of changes, so I didn't go through it with a fine tooth comb, but it looks good to me. I tested it as well and it looks good, but @tonydisera could you test it as well just to make sure. Then we should be able to merge