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

Bug: OBJ File export currently automatically adds GenSmoothNormals when not required #5514

Open
JulianKnodt opened this issue Mar 26, 2024 · 3 comments
Labels
Bug Global flag to mark a deviation from expected behaviour Wavefront-Obj Bugs related to the wavefront format

Comments

@JulianKnodt
Copy link
Contributor

JulianKnodt commented Mar 26, 2024

Describe the bug
When exporting an Obj file, currently GenSmoothNormals is automatically added as a preprocessing step, but an OBJ doesn't need this. I'm not technically sure if it's a bug, but a user should be able to control this behavior.

Expected behavior
If no normals are provided, it shouldn't generate any.

Additional context
I'm also not sure if there is a way to disable this preprocessing step, even though it is automatically added.

It is added near line 162 of Exporter.cpp

@JulianKnodt JulianKnodt added the Bug Global flag to mark a deviation from expected behaviour label Mar 26, 2024
@kimkulling
Copy link
Member

I am wondering why this post-processing step is always on.

@kimkulling
Copy link
Member

Could you show us some code how the stuff gets imported? Special the importer flags are interesting.

Thanks in advance!

@kimkulling kimkulling added the Wavefront-Obj Bugs related to the wavefront format label Mar 28, 2024
@JulianKnodt
Copy link
Contributor Author

What do you mean by code?

This is the previous export:
image

This is the export with smooth normals turned off:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Global flag to mark a deviation from expected behaviour Wavefront-Obj Bugs related to the wavefront format
Projects
Status: No status
Development

No branches or pull requests

2 participants