Skip to content

Conversation

@dmarcos
Copy link
Member

@dmarcos dmarcos commented Mar 20, 2019

No description provided.

@dmarcos
Copy link
Member Author

dmarcos commented Mar 20, 2019

This is ready.

@gfodor
Copy link
Contributor

gfodor commented Mar 22, 2019

qq on this -- is this patch dependent upon anything specific in super-three? we're looking to cherry pick this commit into our A-Frame fork which adds (hacky, not ready for PR) multiview support, but are hesitant to move over to super-three without more testing/validation. Thanks!

@dmarcos
Copy link
Member Author

dmarcos commented Mar 24, 2019

@gfodor super-three is stock THREE + PR waiting to merge upstream. At the moment mrdoob/three.js#16015, mrdoob/three.js#15347 and mrdoob/three.js#15290

Intent is always using stock THREE. If you can spare a comment on those PRs might help expedite merge and put us back on track.

@gfodor
Copy link
Contributor

gfodor commented Mar 25, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants