Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cannot use cubemap in pythreejs #288

Open
kingease opened this issue Oct 22, 2019 · 5 comments
Open

cannot use cubemap in pythreejs #288

kingease opened this issue Oct 22, 2019 · 5 comments

Comments

@kingease
Copy link

when assign scene.background a CubeTexture, there is an error occured:

TraitError: The 'background' trait of a Scene instance must be a valid HTML color, but a value of CubeTexture ...

and in pythreejs, cannot use the cubemap as the way in three.js. because the CubeTextureLoader in pythreejs has no methods like load etc. and I cannot find any reference about this part.

anyone can help?
thanks

@vidartf
Copy link
Member

vidartf commented Oct 22, 2019

The logic for this has not been fully implemented yet. I think one thing we could do would be to add some custom field to the CubeTexture definition (e.g. imageUris), that are only used in the JS constructor, similar to the ImageTexture custom class.

@jiangzhongshi
Copy link

What is the current status of this issue? Thank you. Are there any alternatives or workarounds? Thank you!

@vidartf
Copy link
Member

vidartf commented Mar 5, 2020

As far as I know, no-one has looked at this yet. Please write here if you intended to help implement this :)

@jiangzhongshi
Copy link

Thanks for the reply. I would be happy to make it happen, unfortunately I am really not familiar with javascript.

If it helps anyone in the future, my workaround, at least for inspecting the smoothness through reflection lines, is to use a MeshMatcapMaterial together with DataTexture.

@akaszynski
Copy link
Contributor

So... it's been two years but there's a working example included in #376. Once that's merged we can close this one out.

@akaszynski akaszynski mentioned this issue Apr 21, 2022
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants