Skip to content

Conversation

@vincenzonetti
Copy link
Contributor

Describe your changes

I added a cache to avoid rebuilding the entire mesh topology.
In case the topology is constant a lookup table is used to retrieve the object mesh, avoiding the call to PointDuplicateAccumulator which is expensive

Issue ticket number and link if any

Issure: #2719

Checklist for finalizing the PR

  • [X ] I have performed a self-review of my code
  • I have added tests for new features and bugfixes
  • I have added documentation for new features
  • If it is a modifying the libf3d API, I have updated bindings
  • If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

\ci fast

@mwestphal
Copy link
Member

Interesting, lets discuss the design on discord please :)

@vincenzonetti
Copy link
Contributor Author

@mwestphal @Meakk @snoyer this PR is ready for a review. Thanks!

@mwestphal
Copy link
Member

@Ni-g-3l please review

Copy link
Member

@mwestphal mwestphal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some questions

@vincenzonetti
Copy link
Contributor Author

\ci fast

Copy link
Member

@mwestphal mwestphal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats looks fine to me but this needs a review from @Ni-g-3l

@mwestphal
Copy link
Member

\ci full

@mwestphal mwestphal requested review from Meakk and Ni-g-3l January 4, 2026 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants