iobio / vcf.iobio.io

MIT License
27 stars 11 forks source link

File not loading #19

Closed AlistairNWard closed 8 years ago

AlistairNWard commented 8 years ago

EVA have some files that don't load. I've put them on AWS so they can be checked at:

http://vcf.iobio.io/?vcf=http://s3.amazonaws.com/iobio/eva_test/genomedk.chrY.gatk_indels.r1.EVA.NOT_LOADING.vcf.gz

http://vcf.iobio.io/?vcf=http://s3.amazonaws.com/iobio/eva_test/genomedk.chr22.gatk_indels.r1.EVA.NOT_LOADING.vcf.gz

AlistairNWard commented 8 years ago

Modified file locations:

http://vcf.iobio.io/?vcf=https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load.vcf.gz

http://vcf.iobio.io/?vcf=https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load_2.vcf.gz

http://vcf.iobio.io/?vcf=https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load_3.vcf.gz

AlistairNWard commented 8 years ago

http://vcf.iobio.io/?vcf=https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load.vcf.gz

Loads fine, but there is something odd with the indel distribution (the x axis appears to be missing).

http://vcf.iobio.io/?vcf=https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load_2.vcf.gz

Only contains chromosome 22, fails to load and is riddled with errors.

http://vcf.iobio.io/?vcf=https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load_3.vcf.gz

Only contains chromosome Y, fails to load and is riddled with errors.

AlistairNWard commented 8 years ago

screen shot 2016-06-10 at 9 48 37 am

AlistairNWard commented 8 years ago

Just handle the image problem. Second two files moved to #31

tonydisera commented 8 years ago

I've migrated the chart over to Chase's iobio.viz bar chart object so that outliers can be ignored. The chart is still quite difficult to read.

screenshot 2016-06-10 22 16 32

We could also use Chase's viz component barViewer so that the user could select a region to "zoom" into.

screenshot 2016-06-10 22 23 40

This would require a taller bottom section (+100 px vertical space).

AlistairNWard commented 8 years ago

Looks way better to me. We seem to be using a lot more vertical space for the barViewer than used in bam.iobio. Is it possible to make this a bit smaller?

I'm also seeing an error in the console regarding reading the index. Is this something real?

Uncaught ReferenceError: ignoreErrorMessage is not defined vcf.iobio.js:1024 tabix Error: stderr - Could not load .tbi/.csi index of https://s3.amazonaws.com/iobio/app_testing/vcf_files/fails_to_load.vcf.gz

vcf.iobio.js:1024 tabix Error: prog process exited with code 1

tonydisera commented 8 years ago

Thanks for the feedback! Better?

screenshot 2016-06-15 08 16 01
AlistairNWard commented 8 years ago

I'm happy with it. If we could make the top panel in the 'Insertion & Deletion Lengths' take up more vertical space and squeeze them together a bit more, I don't think that would be a bad thing, but if that can't be done, I'm happy for this to be closed.

tonydisera commented 8 years ago

Yeah, this is probably the best I can do for now. Otherwise, it is hard to select the bounding rectagle on the bottom chart.