search

biocore / Evident-initial-demo

Elucidating sampling effort for microbial analysis studies
7 stars 12 forks source link

Re-center camera not working #12

Closed ElDeveloper closed 11 years ago

ElDeveloper commented 12 years ago

When rotating a PCoA plot and trying to use the Recenter Camera button in the Options tab, the camera is not completely re-centered.

It was mentioned that this issue could be caused given a bug in Three.

meganap commented 12 years ago

Can you link to something describing the bug?

ElDeveloper commented 12 years ago

Using the Fierer et. al. Keyboard demo visualization:

With the dataset as it comes from the demo visualization: first

If you rotate the plot in a random way: second

After using the re-center camera button: third

I would have expected the third picture to look like the first one.

meganap commented 12 years ago

I meant an example of the three.js bug. The way I have to code in evident is that it uses the same line of code it does to set up the camera in the first place, so I don't know what else to change to make it look like the first one.

ElDeveloper commented 12 years ago

Oh I wouldn't know the specifics from three.js, though I recall that @antgonza mentioned this could be because of an outdated version of three.js though I might be wrong.

Probably mentioning the issue on the tree.js github site, opening an issue maybe?

meganap commented 12 years ago

This is a known bug in three.js https://github.com/mrdoob/three.js/issues/821

ElDeveloper commented 11 years ago

Bump, https://github.com/mrdoob/three.js/issues/821 seems to have been fixed in the master repo in https://github.com/mrdoob/three.js/pull/3073 Perhaps worth updating the version of the files within E-vident.

antgonza commented 11 years ago

+1

ElDeveloper commented 11 years ago

Fixed in #40.