Commit 2d8ab9b
MAPSSDK-891 Add Style Version Info to Benchmark and Examples App (#6530)
This PR adds information about the specific version of the style that is
being tested to our benchmark reports. For our benchmarks on iOS and
Android this information can then be used in our Mode reports to
understand which specific version of Standard, Standard Satellite, or
other styles were run in that benchmark. This will help us to understand
how specific modifications to these styles affect our benchmarks. GL JS
[changes](https://github.com/mapbox/mapbox-gl-js-performance-internal/pull/229)
for comparison.
I've tested the full pipeline on several of our Mode charts, and will
update the rest after this change is merged in and we have more data
populated. Samples:
<img width="914" height="96" alt="Screenshot 2025-09-17 at 3 23 33 PM"
src="https://github.com/user-attachments/assets/9f989556-51d2-4778-8c83-96d262ef7f43"
/>
<img width="326" height="98" alt="Screenshot 2025-09-17 at 3 22 53 PM"
src="https://github.com/user-attachments/assets/31482492-9b12-4b13-b4f9-c5853724794c"
/>
Implementation approach:
- Style Info Capture: Added extractStyleInfo() functions to capture
style URL and modification date from loaded map styles during the
benchmark run
- Consistent format: "styleURL - modifiedDate" for benchmark reporting,
with fallbacks to "Unknown" values
- This information is then added to the attributes json for reporting in
Mode, with the specific implementation depending on the platform:
Android Changes:
- Added `extractStyleInfo()` method to MapBenchmark base class and
CreateMapCommand
- Extract this info in CreateMapCommand, StandardStyleBenchmark,
StyleBenchmark
- Added STYLE_VERSION constant to AttributesRule for benchmark metadata
iOS Changes:
- In CreateMapCommand and BaseBenchmark capture style version during map
loading
- Implementation is in MapView+StyleInfo
- Integrated style version extraction in MetricsCommand for benchmark
reporting
Other changes:
- Debug UI Enhancement: Added "Style Info" menu option to
DebugModeActivity (Android) and DebugMapExample (iOS) to display style
details including URL, modification date, and SDK compatibility
<img width="300" alt="Screenshot_20250917_175115"
src="https://github.com/user-attachments/assets/ce1cb4b1-70f1-456e-85d9-3b1f790cea44"
/>
<img width="300" alt="Simulator Screenshot - iPhone 17 - 2025-09-17 at
17 48 19"
src="https://github.com/user-attachments/assets/24e42d0c-301f-4048-b3e2-dfcfd59efc08"
/>
🤖 Partially generated with https://claude.ai/code
cc @mapbox/maps-android
cc @mapbox/maps-ios
cc @mapbox/sdk-ci
---------
Co-authored-by: Release SDK bot for Maps SDK team <[email protected]>
Co-authored-by: Claude <[email protected]>
GitOrigin-RevId: 8346ee78fc10fa02609e6d5decf5455ad89c2b441 parent 5add6ac commit 2d8ab9b
File tree
3 files changed
+87
-1
lines changed- app/src/main
- java/com/mapbox/maps/testapp/examples
- res
- values
3 files changed
+87
-1
lines changedLines changed: 82 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
4 | 5 | | |
5 | 6 | | |
6 | 7 | | |
| |||
18 | 19 | | |
19 | 20 | | |
20 | 21 | | |
| 22 | + | |
| 23 | + | |
21 | 24 | | |
22 | 25 | | |
23 | 26 | | |
| |||
237 | 240 | | |
238 | 241 | | |
239 | 242 | | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
240 | 246 | | |
241 | 247 | | |
242 | 248 | | |
243 | 249 | | |
244 | 250 | | |
245 | 251 | | |
246 | 252 | | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
247 | 295 | | |
248 | 296 | | |
249 | 297 | | |
| |||
342 | 390 | | |
343 | 391 | | |
344 | 392 | | |
345 | | - | |
| 393 | + | |
| 394 | + | |
| 395 | + | |
| 396 | + | |
| 397 | + | |
| 398 | + | |
| 399 | + | |
| 400 | + | |
| 401 | + | |
| 402 | + | |
| 403 | + | |
| 404 | + | |
| 405 | + | |
| 406 | + | |
| 407 | + | |
| 408 | + | |
| 409 | + | |
| 410 | + | |
| 411 | + | |
| 412 | + | |
| 413 | + | |
| 414 | + | |
| 415 | + | |
| 416 | + | |
| 417 | + | |
| 418 | + | |
| 419 | + | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
| 425 | + | |
| 426 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
48 | 52 | | |
49 | 53 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
| 133 | + | |
133 | 134 | | |
134 | 135 | | |
135 | 136 | | |
| |||
0 commit comments