Skip to content
This repository has been archived by the owner on Dec 3, 2018. It is now read-only.

Profiling tools #20

Closed
yhahn opened this issue May 8, 2013 · 4 comments
Closed

Profiling tools #20

yhahn opened this issue May 8, 2013 · 4 comments
Assignees

Comments

@yhahn
Copy link
Member

yhahn commented May 8, 2013

Brainstormed some concepts on the train to help designers determine how fast/slow their styles are in terms of rendering speed.

Will prototype.

cc @ian29 @ajashton @springmeyer

@ghost ghost assigned yhahn May 8, 2013
@springmeyer
Copy link
Contributor

Great, curious to hear your ideas. Starting really high level I think will be good - like just providing quick feedback on aggregate tile load times (maybe reporting a small sparkline of last 10 saves?) or a quick hover per tile mode. I logged more low level ideas previously at https://github.com/mapbox/mapbox-streets/issues/368

@yhahn
Copy link
Member Author

yhahn commented May 8, 2013

y agreed

@yhahn
Copy link
Member Author

yhahn commented May 13, 2013

I've started some experiments with this in the renderstats branch. Currently focusing on measuring render time (ie. excludes time to download/cache hit vector tiles, map compile/load time, etc.) as it's the most relevant stat for the designer to tune for.

The code so far stores min, max, and avg for tile draws. I wanted to see if this stat would yield consistent enough results to be useful. So far, it looks like yes, with max sometimes being an outlier (e.g. you start watching a youtube video while your map is loading === single max outlier).

stats-1

stats-2

stats-3

Next question for me is how it feels to use these stats and whether they can actually be used during the design process. Would love for you guys to try it out @ajashton @ian29 @springmeyer


ps. I also briefly tried storing stats on a per-tile basis and quickly backed out. My brief notes: a lot of overhead, difficult to convey data in an actionable manner, or rather, possibly too much detail overall to be useful.

@yhahn
Copy link
Member Author

yhahn commented May 14, 2013

Merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants