search

rega-cev / phylogeotool

4 stars 3 forks source link

The bars in histograms seem to be grouped per 3 bars. #6

Closed plibin closed 7 years ago

plibin commented 7 years ago

There should be no grouping, simply a uniform placement.

e.g.: http://phylogeotool.gbiomed.kuleuven.be/dengue/, attribute="Collection date"

Ewout1988 commented 7 years ago

They are not grouped per 3 bars. Everything is displayed uniformly e.g. http://phylogeotool.gbiomed.kuleuven.be/euresist/root/1

GuyBaele commented 7 years ago

Agreed, this doesn't seem to be a problem. But why is 'Place your sequence' disabled in the dengue example?

Ewout1988 commented 7 years ago

Because the Dengue tool was deployed recently. I haven't had the time nor the resources to test if the pplacing works with adding new Dengue sequences. So it was better to turn this feature off for now.

plibin commented 7 years ago

On 25 Apr 2017, at 15:20, Guy Baele notifications@github.com<mailto:notifications@github.com> wrote:

Agreed, this doesn't seem to be a problem.

I find this kind of grouping very strange in an histogram, and got the sample idea from a few colleagues here, so I think this needs to be fixed.

But why is 'Place your sequence' disabled in the dengue example?

I will make a new issue for this.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/rega-cev/phylogeotool/issues/6#issuecomment-297027337, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFLrgtCPnVMP_fBxAY-Y3IeyjnjgmvStks5rzfMNgaJpZM4NHdpS.

{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/rega-cev/phylogeotool","title":"rega-cev/phylogeotool","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/rega-cev/phylogeotool"}},"updates":{"snippets":[{"icon":"PERSON","message":"@GuyBaele in #6: Agreed, this doesn't seem to be a problem.\r\nBut why is 'Place your sequence' disabled in the dengue example?"}],"action":{"name":"View Issue","url":"https://github.com/rega-cev/phylogeotool/issues/6#issuecomment-297027337"}}}

GuyBaele commented 7 years ago

Ok, then this issue can be closed and I'll create a new one to test pplacer with the Dengue deployment.

plibin commented 7 years ago

did this already On 25 Apr 2017, at 15:29, Guy Baele notifications@github.com<mailto:notifications@github.com> wrote:

Ok, then this issue can be closed and I'll create a new one to test pplacer with the Dengue deployment.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/rega-cev/phylogeotool/issues/6#issuecomment-297030012, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFLrguuvdwdZUQBnxxhxmKN8Lz6FKhRbks5rzfU9gaJpZM4NHdpS.

{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/rega-cev/phylogeotool","title":"rega-cev/phylogeotool","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/rega-cev/phylogeotool"}},"updates":{"snippets":[{"icon":"PERSON","message":"@GuyBaele in #6: Ok, then this issue can be closed and I'll create a new one to test pplacer with the Dengue deployment."}],"action":{"name":"View Issue","url":"https://github.com/rega-cev/phylogeotool/issues/6#issuecomment-297030012"}}}