Skip to content

[MISC] Remove fast_simplification submodule.#1497

Merged
duburcqa merged 8 commits intoGenesis-Embodied-AI:mainfrom
Kashu7100:remove-fast-simplification
Aug 5, 2025
Merged

[MISC] Remove fast_simplification submodule.#1497
duburcqa merged 8 commits intoGenesis-Embodied-AI:mainfrom
Kashu7100:remove-fast-simplification

Conversation

@Kashu7100
Copy link
Collaborator

@Kashu7100 Kashu7100 commented Aug 4, 2025

Description

  • Remove 'fast_simplication' submodule to rather rely on pre-compiled binaries from pyvista available on PyPI.

Motivation and Context

Having to compile fast_simplification submodule from source when building wheels for Genesis forces to distribute platform-specific pre-compiled binaries on PyPI. This is tedious and not worth the effort considering how limited is the benefit of maintaining our own version of 'fast_simplication' library. It is only exposing lossless simplification, which could be easily contributed upstream directly.

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.

@Kashu7100 Kashu7100 requested a review from duburcqa August 4, 2025 20:58
@duburcqa duburcqa changed the title [MISC] remove fast_simplification from ext [MISC] Remove fast_simplification from ext Aug 4, 2025
duburcqa
duburcqa previously approved these changes Aug 4, 2025
@duburcqa duburcqa enabled auto-merge (squash) August 4, 2025 23:09
@duburcqa duburcqa changed the title [MISC] Remove fast_simplification from ext [MISC] Remove fast_simplification submodule. Aug 4, 2025
@duburcqa duburcqa disabled auto-merge August 5, 2025 00:35
@duburcqa duburcqa merged commit a7b089b into Genesis-Embodied-AI:main Aug 5, 2025
17 checks passed
YilingQiao pushed a commit to YilingQiao/Genesis that referenced this pull request Aug 22, 2025
winnieyangwannan pushed a commit to winnieyangwannan/Genesis that referenced this pull request Sep 14, 2025
Kashu7100 added a commit to Kashu7100/Genesis that referenced this pull request Jan 26, 2026
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.

2 participants