Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Fix Instancing Component #7602

Merged
merged 3 commits into from
Feb 20, 2023
Merged

Fix Instancing Component #7602

merged 3 commits into from
Feb 20, 2023

Conversation

dinomut1
Copy link
Member

Fixes instancing component:
-Grass is rendered correctly with the logarithmic depth buffer
-Node editor reimplemented with hyperflux
-Instancing component reimplemented with defineComponent schema

Copy link
Member

@HexaField HexaField left a comment

Choose a reason for hiding this comment

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

Nice work, let's consider doing this in a follow up

grassTexture: 'https://resources-dev.etherealengine.com/assets/grass/blade_diffuse.jpg',
alphaMap: 'https://resources-dev.etherealengine.com/assets/grass/blade_alpha.jpg',
grassTexture: {
src: 'https://resources-dev.etherealengine.com/assets/grass/blade_diffuse.jpg',
Copy link
Member

Choose a reason for hiding this comment

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

We should be sourcing these from the repo rather than the engine in case they are unreachable

Copy link
Member

Choose a reason for hiding this comment

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

@dinomut1 dinomut1 added this pull request to the merge queue Feb 20, 2023
Merged via the queue into dev with commit a7b713a Feb 20, 2023
@dinomut1 dinomut1 deleted the fix-instancing branch February 20, 2023 00:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants