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

Mesh shading looks weird #5011

Closed
normanrz opened this issue Dec 29, 2020 · 5 comments · Fixed by #5326
Closed

Mesh shading looks weird #5011

normanrz opened this issue Dec 29, 2020 · 5 comments · Fixed by #5326
Assignees

Comments

@normanrz
Copy link
Member

The shading/lighting of imported meshes looks weird; as if there are multiple colors blended in:
Screenshot 2020-12-29 at 11 28 50

@MichaelBuessemeyer
Copy link
Contributor

@normanrz My specific task here is what exactly? To change the shader / the way the mesh is rendered?
As far as I can remember we had the idea of doing this iteratively and that the first version should be quite simple. As I can remember you already had something in mind for that, right @philippotto?

Just some comment:
My guess why the shading looks this way is because the normals are rendered for the mesh. (See Normal Mapping).

@normanrz
Copy link
Member Author

The shading of the on-demand rendered meshes is good. I think the uploaded meshes should be shaded in the same way. Ideally, each uploaded mesh would get its own color (maybe cycle through the tree color list?).

@philippotto
Copy link
Member

As I can remember you already had something in mind for that, right @philippotto?

Yes, that's what I meant:

The shading of the on-demand rendered meshes is good. I think the uploaded meshes should be shaded in the same way.

@MichaelBuessemeyer
Copy link
Contributor

For the record: The "on-demand rendered meshes" are the rendered isosurfaces.

@normanrz
Copy link
Member Author

For the record: The "on-demand rendered meshes" are the rendered isosurfaces.

They are both "meshes" and both "isosurfaces" ;-)

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

Successfully merging a pull request may close this issue.

3 participants