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

[core] Wireframe debug mode #4359

Merged
merged 2 commits into from
May 23, 2016
Merged

[core] Wireframe debug mode #4359

merged 2 commits into from
May 23, 2016

Conversation

brunoabinader
Copy link
Member

@brunoabinader brunoabinader commented May 21, 2016

Together with MapDebugOptions::Collision, provides wireframe rendering output so all drawn objects are visible. This mode ignores the line width, background and fill colors for better visualization.

/cc @mapbox/gl

@brunoabinader brunoabinader added the performance Speed, stability, CPU usage, memory usage, or power usage label Mar 17, 2016
@brunoabinader brunoabinader changed the title Wireframe debug mode [core] Wireframe debug mode Mar 18, 2016
brunoabinader added a commit that referenced this pull request May 21, 2016
Together with MapDebugOptions::Collision, provides "wireframe" rendering
output so all drawn objects are visible. This mode ignores the line
width, background and fill colors for better visualization.

Fixes #4359.
@brunoabinader
Copy link
Member Author

brunoabinader commented May 21, 2016

Example output:

wireframe

👀 @kkaefer @jfirebaugh @tmpsantos @1ec5

@1ec5
Copy link
Contributor

1ec5 commented May 21, 2016

Awesome – this is the mode @kkaefer requested in #4669, right? It’ll be pretty easy to hook this up to the iOS and OS X SDKs now that there’s a MapDebugOptions option for it.

@brunoabinader
Copy link
Member Author

@1ec5 I think this is what @kkaefer mentioned - https://github.com/mapbox/mapbox-gl-native/blob/master/platform/default/glfw_view.cpp#L483-L520 - which provides a different output - but can be added as a new MapDebugOptions as such.

@1ec5
Copy link
Contributor

1ec5 commented May 22, 2016

22ef615f360bb5e0ea58491620f0fc59da8f491b exposes this new mode in the iOS and OS X SDKs and their demo applications.

@1ec5 1ec5 added iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS labels May 22, 2016
@brunoabinader
Copy link
Member Author

Thank you @1ec5!

brunoabinader and others added 2 commits May 23, 2016 11:50
Together with MapDebugOptions::Collision, provides "wireframe" rendering
output so all drawn objects are visible. This mode ignores the line
width, background and fill colors for better visualization.

Fixes #4359.
Added an option to MGLMapDebugMaskOptions for the new wireframe debugging mode and UI in the demo applications for toggling it.
@brunoabinader brunoabinader force-pushed the brunoabinader-wireframe branch from 22ef615 to 2865e7d Compare May 23, 2016 08:59
@kkaefer
Copy link
Member

kkaefer commented May 23, 2016

@brunoabinader stencil mask mode is different from the wireframe mode; the stencil mode shows the checkerboard-style pattern that we're using to clip tiles.

@brunoabinader
Copy link
Member Author

@kkaefer yes, that's exactly what I mentioned to @1ec5 in the last comment :) We should move the stencil mask code to core and add a new enum for it in MapDebugOptions.

@brunoabinader brunoabinader merged commit 2865e7d into master May 23, 2016
@brunoabinader brunoabinader deleted the brunoabinader-wireframe branch May 23, 2016 09:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS performance Speed, stability, CPU usage, memory usage, or power usage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants