Skip to content

[FEATURE] Enabling rendering plane geometry one-sided. #1396

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

Merged
merged 26 commits into from
Jul 15, 2025

Conversation

LeonLiu4
Copy link
Contributor

@LeonLiu4 LeonLiu4 commented Jul 10, 2025

create_plane() now accepts double_sided (default false), returning a single-sided quad with the same footprint/UVs.

Addresses #1112

Two sided:

Screen.Recording.2025-07-11.at.8.54.52.PM.mov

One sided:

Screen.Recording.2025-07-11.at.8.55.35.PM.mov

@duburcqa duburcqa changed the title Feature/one sided plane [FEATURE] Add support of rendering plane geometry one-sided. Jul 14, 2025
@duburcqa duburcqa changed the title [FEATURE] Add support of rendering plane geometry one-sided. [FEATURE] Enabling rendering plane geometry one-sided. Jul 14, 2025
@duburcqa
Copy link
Collaborator

Is this patch all that was needed to fix one-sided plane?! That would be awesome :)

@LeonLiu4
Copy link
Contributor Author

Is this patch all that was needed to fix one-sided plane?! That would be awesome :)

Unfortunately that patch created a mesh that was not a solid volume causing there to be errors with stuff like collisions. However I have rectified that by differentiating between a visual and collision mesh

@duburcqa duburcqa enabled auto-merge (squash) July 15, 2025 20:51
@duburcqa
Copy link
Collaborator

@LeonLiu4 Can you confirm this PR is ready to merge and the videos in description are still valid?

@LeonLiu4
Copy link
Contributor Author

@LeonLiu4 Can you confirm this PR is ready to merge and the videos in description are still valid?

Yes the videos are correct and its ready to merge

@duburcqa duburcqa merged commit 2bff5ef into Genesis-Embodied-AI:main Jul 15, 2025
13 checks passed
@LeonLiu4 LeonLiu4 deleted the feature/one-sided-plane branch July 16, 2025 01:44
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.

3 participants