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

HARP-17373: Fix shader compilation in Safari 15 on MacOS Monterrey. #2337

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

atomicsulfate
Copy link
Collaborator

@atomicsulfate atomicsulfate commented Nov 22, 2021

Shader compilation errors using Metal through ANGLE compatibility layer were caused by the uniform names 'diffuse' and 'bitangent'.

Signed-off-by: Andres Mandado andres.mandado-almajano@here.com

@atomicsulfate atomicsulfate force-pushed the HARP-17373 branch 2 times, most recently from f392610 to 19fe4f9 Compare November 23, 2021 15:12
@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #2337 (e852e5a) into master (6339034) will decrease coverage by 0.02%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2337      +/-   ##
==========================================
- Coverage   68.39%   68.37%   -0.03%     
==========================================
  Files         319      319              
  Lines       27915    27915              
  Branches     6250     6250              
==========================================
- Hits        19093    19086       -7     
- Misses       8822     8829       +7     
Impacted Files Coverage Δ
@here/harp-lines/lib/Lines.ts 76.70% <ø> (ø)
@here/harp-mapview/lib/geometry/TileGeometry.ts 6.85% <0.00%> (ø)
...re/harp-materials/lib/HighPrecisionLineMaterial.ts 38.23% <0.00%> (ø)
...e/harp-materials/lib/HighPrecisionPointMaterial.ts 20.45% <ø> (ø)
@here/harp-materials/lib/SolidLineMaterial.ts 57.48% <50.00%> (ø)
@here/harp-mapview/lib/geometry/SolidLineMesh.ts 98.91% <100.00%> (ø)
@here/harp-materials/lib/CirclePointsMaterial.ts 100.00% <100.00%> (ø)
@here/harp-mapview/lib/VisibleTileSet.ts 78.20% <0.00%> (-1.58%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6339034...e852e5a. Read the comment docs.

…onterrey.

Shader compilation errors were caused by the uniform names 'diffuse' and
'bitangent'.

Signed-off-by: Andres Mandado <andres.mandado-almajano@here.com>
@AndriiHeonia AndriiHeonia merged commit bc9bf92 into master Nov 25, 2021
@AndriiHeonia AndriiHeonia deleted the HARP-17373 branch November 25, 2021 13:59
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