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

Support recent versions of THREE #12

Open
sguimmara opened this issue May 27, 2024 · 3 comments
Open

Support recent versions of THREE #12

sguimmara opened this issue May 27, 2024 · 3 comments

Comments

@sguimmara
Copy link

Recent version of THREE.js have some breaking changes that impact this loader. Here is a short list of changes that might be of interest in cityjson-threejs-loader:

@liberostelios
Copy link
Contributor

Hi @sguimmara.

Thank you so much for reporting this 🙌 And apologies for the late response.

Indeed, the library is now breaking with recent versions of three.js. I have them in my radar and planning on addressing them soon.

@liberostelios
Copy link
Contributor

@sguimmara

de6628b is addressing the issue regarding changes in colorspace management. Therefore the library is now expected to work with v165.

I have not spotted any issues with the the other two issues: light changes should affect the setup of the renderer and scene on the client side, rather than this library. Am I missing something? Any suggestions?

@sguimmara
Copy link
Author

sguimmara commented Jun 19, 2024

For the color space change, you might want to check that colors set on the materials generated by the library look the same before and after the upgrade. If they don't, it means that their color space is not correct. We had this issue when working with Giro3D

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

No branches or pull requests

2 participants