Commit Graph

15250 Commits

Author SHA1 Message Date
Thiago Marcos P. Santos
d13562252d [metrics] Isolate binary size metrics
Will give us a better view of what changed and by how much. Otherwise
the test stops at the first failure.
2019-11-28 12:34:22 +02:00
Thiago Marcos P. Santos
09b724ea33 [render-test] Fix regex matching the directory name
This code was matching `style.json` instead of `metrics.json` if
the `style.json` was contained inside `metrics/.../style.json`.
2019-11-27 18:32:58 +02:00
Thiago Marcos P. Santos
2a65301f09 [build] Get the bots enforcing the binary size
Release bots will build and persist the build targets other workspaces.

A bot will collect all the binaries and check the size.
2019-11-27 18:32:58 +02:00
Thiago Marcos P. Santos
cf160a24d0 [build] Install some of our targets
This will make it easier to measure the binary size because we can
strip the binaries from symbols in a portable way.
2019-11-27 18:32:58 +02:00
Thiago Marcos P. Santos
8d163c159f [metrics] Add binary size metrics
This test will check the size of the tools and shared libraries
built from previous bots in a single run.

We will have +/- 1% acceptable variance before the test fails.
2019-11-27 18:32:58 +02:00
Thiago Marcos P. Santos
91d83accde [tests] Increase test tolerance
This test has been flaky on mac.
2019-11-27 18:32:58 +02:00
Mikhail Pozdnyakov
2b6a0e48ee [test runner] Enable test results update from cmd line
Added the following command line argument:
```
-u[update], --update=[update]     Test results update mode.
Supported values are: "default", "platform", "metrics"
```
2019-11-27 18:19:10 +02:00
Langston Smith
c35f64a2b5 [mobile] README update about mobile work being moved to new repos (#15958) 2019-11-27 07:18:51 -08:00
Alexander Shalamov
2f8501043c [test-runner] Move probe related command line args to manifest
- Move probe related command line arguments to manifest, example:

  "probes": ["probeGFX", "probeNetwork", "probeMemory"],
  "metric_path": "metrics/linux"

- Use manifest file name as a postfix for a result page
- Move injected probes 'begin' section before map object creation
- Generalize artifact storing shell script
2019-11-27 16:43:33 +02:00
Łukasz Paczos
f10097640a [android] do not fail on tautological-constant-compare error 2019-11-27 14:41:52 +01:00
zmiao
11321497da [render-test] Add metric_paths into Manifest (#15973)
* [render-test]Add expected metrics path in manifest file

* seperate metrics.json and expected.png

* fix expected path parsing

* keep expectation path if update is enabled
2019-11-27 11:08:55 +02:00
Mikhail Pozdnyakov
5c271dba24 [core] Update Source.GeoJSONSourceTilesAfterDataReset unit test 2019-11-26 16:35:17 +02:00
Mikhail Pozdnyakov
b6fbd1221e [core] Clear tiles when GeoJSON data changes
In tile and static mode clear tiles when data changes
in order to avoid render tests being flaky.
2019-11-26 16:35:17 +02:00
Sudarsana Babu Nagineni
0f3eede6a6 [build] Add an option to include platform specific implementation
Add an option to include platform specific implementation sources
to mbgl-core.
2019-11-26 16:23:36 +02:00
Konstantin Käfer
3600dd8a1a [core] fix rendering of dashed lines with round caps
In https://github.com/mapbox/mapbox-gl-native/pull/15862, we introduced individual textures for line dash patterns to eliminate atlas overflows. Unfortunately, this broke dashed lines that had round caps (dashed lines with straight caps still rendered correctly). Line pattern textures for round caps were now using 256×15 pixel textures.

The OpenGL ES 2.0 spec, section 3.8.2 states:

    Calling a sampler from a fragment shader will return `(R,G,B,A) = (0,0,0,1)` if any of the following conditions are true:
    […]
    - A two-dimensional sampler is called, the corresponding texture image is a non-power-of-two image […], and either the texture wrap mode is not `CLAMP_TO_EDGE`, or the minification filter is neither `NEAREST` nor `LINEAR`.
    […]

This means that texture lookups won't work for NPOT textures unless they use `GL_CLAMP_TO_EDGE`. We're using `GL_CLAMP_TO_EDGE` for the vertical direction, but GL_REPEAT for the horizontal direction, which means that we need a power-of-two texture for our line dash patterns to work on OpenGL ES 2.0 conforming implementations. Fortunately, this just means changing the height from 15 to 16, and from 30 to 32, so we don't waste many pixels.
2019-11-22 16:36:17 +01:00
Alexander Shalamov
17a5441768 [test-runner] Add render test runner options for injecting probes
This allows injection of memory, gfx and network probes before and
after operations defined in render test are run.
2019-11-22 15:29:25 +02:00
zmiao
a02ed76d5b [render-test] Adding mvt-fixture to app asset content (#15955) 2019-11-21 17:22:40 +02:00
Thiago Marcos P. Santos
b3fc469b09 [tests] Disable some metrics recording on Firebase
We are not using them, attempt to make the tests run faster.
2019-11-21 15:31:59 +02:00
Thiago Marcos P. Santos
d6eeea1909 [build] Build Android release when running render tests 2019-11-21 15:31:59 +02:00
Juha Alanen
a1243ac1ea [ios, macos] Changelog entry for network request priority change (#15950) 2019-11-21 14:39:47 +02:00
Juha Alanen
bb44ad232b [android] Changelog entry for network request priority change (#15950) 2019-11-21 14:39:47 +02:00
Juha Alanen
d777ba54bd [test] Add test for priority of online requests for expired resources 2019-11-21 14:39:47 +02:00
Juha Alanen
bdd7e53078 [core] Set the priority of online requests for existing resource to low 2019-11-21 14:39:47 +02:00
Juha Alanen
69d980cdc1 [core] Add new method to DefaultFileSource
Add setMaximumConcurrentRequests() method for setting it on the
OnlineFileSource.
2019-11-21 14:39:47 +02:00
Juha Alanen
451e6d40fa [core] Notify requests about network reachability only when coming back online 2019-11-21 14:39:47 +02:00
Julian Rex
21af0ff487 [ios] Refactor host checks (#15948) 2019-11-20 12:44:39 -05:00
zmiao
03b7fe3326 [render-test] Config CircleCi to run RenderTestApp on Firebase with Pixel4 (#15951)
This reverts commit d2413d4ffffd087b352b6e4edd6e6322a311bc07.
2019-11-20 17:56:19 +02:00
Thiago Marcos P. Santos
656bc9c8bb [build] Run Valgrind instead of Memory Sanitizer
Memory Sanitizer requires all the dependencies, including
libstdc++ to be compiled with Memory Sanitizer, otherwise
we will get tons of false positives.
2019-11-20 13:56:48 +02:00
zmiao
1b58e87f15 [render-test] Wrap test data inside RenderTestRunner App + Add a new test app (#15887)
* [render-test] Wrap test resources inside app

* [render-test] Add test app

* fix test app failure

* [render-test]add callback + add javaObjectWrapper
2019-11-19 20:45:30 +02:00
Mikhail Pozdnyakov
abdd32d11f [ios][android] Change log entries for #15941 2019-11-19 16:08:35 +02:00
Roman Kuznetsov
0ea086ecf8 [core] Remove default move operator for RenderTile 2019-11-19 16:06:18 +02:00
Mikhail Pozdnyakov
9f24115d13 [core] Identation and build fixes 2019-11-19 15:03:28 +02:00
Sergey Yershov
37b332899b [core] Fix incorrect resizing of TileCache 2019-11-19 15:03:28 +02:00
Mikhail Pozdnyakov
15e1d35488 [core] Add Style.SourceImplsOrder test 2019-11-19 14:39:52 +02:00
Mikhail Pozdnyakov
986465e015 [core] Introduce style::CollectionWithPersistentOrder
So that we do not unnecessarily remove and re-add
sources and images if their order has been modified
in the updated style.
2019-11-19 14:39:52 +02:00
Thiago Marcos P. Santos
b3fdd2c762 [build] Check if mapbox-gl-js is pinned to master 2019-11-19 13:42:16 +02:00
Thiago Marcos P. Santos
f73467786d [build] Enable more warnings on the next build
Make compilation more pedantic.
2019-11-15 23:38:47 +02:00
Ander Conselvan de Oliveira
4c1d03c257 [core] Fix MapSnapshotter build failure on Windows
MSVC implementation of std::promise is buggy and only works with types
that can be default-constructed. To avoid a compilation failure in the
instantiation of ask() inside MapSnapshotter::getRegion(), which creates
a std::promise<LanLngBounds>, make LatLngBounds' default constructor
public.
2019-11-13 13:31:01 +02:00
Mikhail Pozdnyakov
089ff8e2b3 No firebase skip for commits touching paths other than darwin-specific 2019-11-13 10:57:07 +02:00
Mikhail Pozdnyakov
f12567ef8c [android] Add change log entry 2019-11-12 17:24:50 +02:00
Mikhail Pozdnyakov
ed5ed8f083 [android] Add OfflineRegion.deleteAndSkipPackDatabase 2019-11-12 17:24:50 +02:00
Mikhail Pozdnyakov
8451633c17 [android] Consolidate exception handling code in OfflineManager 2019-11-12 17:24:50 +02:00
Mikhail Pozdnyakov
2edbe916ab [android] Add OfflineManager.packDatabase() API 2019-11-12 17:24:50 +02:00
Mikhail Pozdnyakov
f567a79dd8 [core] Extend DefaultFileSource API
Add `packDatabase()` method and `bool pack` argument to the `deleteOfflineRegion()` method.
2019-11-12 17:24:50 +02:00
Mikhail Pozdnyakov
615e095ed6 [core] OfflineDatabase pack API
- introduce `OfflineDatabase::pack()` standing for incremental vacuum
- make pack optional at offline region deletion
- update `OfflineDatabase.DeleteRegion` test accordingly to the items above
2019-11-12 17:24:50 +02:00
Dane Springmeyer
c1b2e8c9a0 fix -Wfor-loop-analysis and -Wreorder warnings (#15901) 2019-11-12 06:48:37 -08:00
Dane Springmeyer
49456e3fc1 remove duplicate formatNumber (#15908) 2019-11-12 06:47:43 -08:00
Alexander Shalamov
85c93e7e83 [android] Update unit tests that use resolvedImage type 2019-11-12 16:41:56 +02:00
Mikhail Pozdnyakov
ee38730eed [core] Add Source.GeoJSONSourceTilesRemainAfterDataReset unit test 2019-11-12 15:12:31 +02:00
Mikhail Pozdnyakov
2246cf7cc7 [ios][android] Add change log entries 2019-11-12 15:12:31 +02:00