Skip to content

Internal support for parsing texture and normal indices in OBJ loader #8835

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

Closed
wants to merge 4 commits into from

Conversation

mehull-26
Copy link
Contributor

Summary of Changes

This PR adds the ability to parse normals and texture coordinates from .obj files using the existing face parsing logic. This is currently internal-only, and not exposed in the documented API or parameter sets.

The goal is to fully integrate OBJ normal and texture support in a future PR, once the writer logic is updated, and a broader set of test cases are validated.

  • Internal parsing logic implemented
  • Unit test added for face parsing with v/vt/vn syntax
  • Not exposed to public interface yet

Why not public yet?

  • Writer does not yet support output of texture and normal attributes.
  • Further edge case testing is necessary before public exposure.

Once these conditions are met, the feature will be promoted to the public API.

Release Management

  • Affected package(s): Stream Support
  • Issue(s) solved (if any): partially worked on #8344
  • Feature/Small Feature (if any): added capability of reading normals and texture coordinates in OBJ file format (not yet exposed to public API though, only kept to internal::read_obj() )
  • Link to compiled documentation (obligatory for small feature) Not yet exposed to the public API to write documentation for
  • License and copyright ownership: Same as CGAL

mehull-26 and others added 3 commits April 5, 2025 17:38
added the support of reading normals and texture coordinates in OBJ.h internal::read_OBJ(). changes not pushed yet to the user to the external CGAL API
@mehull-26 mehull-26 changed the title Internal support for parsing texture and normal indices in OBJ loader and lil typo Internal support for parsing texture and normal indices in OBJ loader Apr 8, 2025
@mehull-26 mehull-26 closed this Apr 21, 2025
@mehull-26
Copy link
Contributor Author

mehull-26 commented Apr 22, 2025

Updated it with a cleaner PR : #8857

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 this pull request may close these issues.

1 participant