-
-
Notifications
You must be signed in to change notification settings - Fork 335
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
MLNCustomStyleLayer + Metal example #2041
Conversation
stefankarschti
commented
Jan 19, 2024
•
edited
Loading
edited
- Rename MLNOpenGLStyleLayer to MLNCustomStyleLayer
- implement example Metal Custom Layer
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2041 +/- ##
==========================================
+ Coverage 58.43% 58.46% +0.03%
==========================================
Files 569 569
Lines 28059 28047 -12
Branches 11209 11197 -12
==========================================
+ Hits 16396 16399 +3
+ Misses 4151 4146 -5
+ Partials 7512 7502 -10 ☔ View full report in Codecov by Sentry. |
Bloaty Results (iOS) 🐋Compared to main
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results-ios/pr-2041-compared-to-main.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great!
There are still some references to OpenGL in MLNCustomStyleLayer.mm
(in the comments).
MLNOpenGLLayerHost
should also be renamed?
Can LimeGreenStyleLayer.mm
be updated to work with Metal?
Could you add an entry to the changelog in I can take care of this one.platform/ios/CHANGELOG.md
?
Bloaty Results 🐋Compared to main
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-2041-compared-to-main.txtCompared to d387090 (legacy)
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-2041-compared-to-legacy.txt |
for more information, see https://pre-commit.ci
…libre/maplibre-gl-native into mlncustomstylelayer-refactor
I'm surprised there are no merge conflicts after #1948 merged 🤔 . |
…into mlncustomstylelayer-refactor
Oh but there are plenty... I'm in the process of resolving them. 😄 |
for more information, see https://pre-commit.ci
…into mlncustomstylelayer-refactor