Skip to content
This repository has been archived by the owner on Jul 20, 2020. It is now read-only.

Use BufferGeometry as a resource? #105

Closed
oortlieb opened this issue Sep 7, 2016 · 9 comments
Closed

Use BufferGeometry as a resource? #105

oortlieb opened this issue Sep 7, 2016 · 9 comments

Comments

@oortlieb
Copy link

oortlieb commented Sep 7, 2016

I'm putting an application together that requires a number of geometry resources. Some of the imported geometries are fairly complex (500k vertices after merging). Three.js hangs when trying to convert these large BufferGeometry objects into Geometries, which I can place into resources.

Is there a way to add BufferGeometries to resources? Or is there an alternative to using Three.js's Geometry.fromBufferGeometry method?

@toxicFork
Copy link
Collaborator

Hi @oortlieb , I will need to implement that, for now as a workaround you can create the mesh manually and add into your groups or scenes e.g.

<scene ref={mySceneRef} > 
    ...

and on componentDidMount:

myScene.add(new THREE.Mesh(myBufferGeometry, myMaterial));

@oortlieb
Copy link
Author

@toxicFork Thanks for the info. Any estimate as to the level of effort and timeline for getting BufferGeometry support added? If it's not on your short-term road map, I might give it a shot myself.

@toxicFork
Copy link
Collaborator

I have done a simple implementation in 306e777 which will allow you to create a buffer geometry and set the 4 attributes, when I have more time ( should be able to get back to work on more r3r stuff soon :D ) I'll be able to fix some other bugs and cut a new release with it :)

Unfortunately no timeline since I can only work on this after work or in weekends when I am not occupied with other things

@oortlieb
Copy link
Author

Looks like it works -- thanks a ton!

@martunta
Copy link
Contributor

is this now part of the release? There is nothing in the docs. Or I should get this specific commit? I have exactly the same problem as @oortlieb

@toxicFork
Copy link
Collaborator

Ah I haven't merged it to master yet but it should be in for the next release

@martunta
Copy link
Contributor

cool. can't wait. When is next release scheduled?

@toxicFork
Copy link
Collaborator

Ah sorry I thought I had replied earlier... I was aiming for a release today but unfortunately I did not have much time. Current ETA for a release is sometime this week ( latest Thursday when I should have more time in the evening :) )

@toxicFork
Copy link
Collaborator

I should be able to do a release tomorrow evening :)

On Sat, 22 Oct 2016, 20:28 Martins Untals, notifications@github.com wrote:

cool. can't wait. When is next release scheduled?


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#105 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0iLb7kY9wqntc9zQbuq_sTIuthQCj3ks5q2mPtgaJpZM4J3Vui
.

toxicFork pushed a commit that referenced this issue Oct 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants