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

Disable depth in heatmap offscreen rendering. #16252

Merged
merged 1 commit into from
Mar 2, 2020

Conversation

astojilj
Copy link
Contributor

@astojilj astojilj commented Mar 1, 2020

It triggers Metal depth stencil validation error since depth test is enabled with no depth attachment.

It triggers Metal depth stencil validation error since depth test is enabled with no depth attachment.
@astojilj astojilj requested review from alexshalamov and zmiao March 1, 2020 09:04
Copy link
Contributor

@alexshalamov alexshalamov left a comment

Choose a reason for hiding this comment

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

lgtm, the change is tested by existing heatmap layer render tests, right?

@astojilj
Copy link
Contributor Author

astojilj commented Mar 1, 2020

lgtm, the change is tested by existing heatmap layer render tests, right?

In addition to render tests run here on CI, Metal tests are run separately. Not sure I understood.

@astojilj astojilj merged commit 42469cd into master Mar 2, 2020
@astojilj astojilj deleted the astojilj-depth-heatmap-offscreen branch March 2, 2020 10:20
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.

3 participants