Skip to content
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

Performance hit on HTML rig setting labels options #235

Open
nvkelso opened this issue Jan 17, 2017 · 1 comment
Open

Performance hit on HTML rig setting labels options #235

nvkelso opened this issue Jan 17, 2017 · 1 comment

Comments

@nvkelso
Copy link
Member

nvkelso commented Jan 17, 2017

Picking up the conversation in 1e07f8e#commitcomment-20493215...

@bcamper said:

I think before we cut a release of this update, we are going to need to find a way to 
drive much more of this in the scene file. Especially with conditionals, it's moving too 
much styling logic into an external file, which will make it unwieldy to support in cases 
like Mapzen.js, and even more so native SDKs. I also observed noticeable performance 
impact from re-running these global substitutions on start-up (they already get applied 
in the scene file, so immediately re-applying them introduces extra overhead).
@nvkelso
Copy link
Member Author

nvkelso commented Jan 17, 2017

The release proposal on the table is each YAML file per basemap style label variant would have the globals set in the YAML file, as has been done previously using the older YAML anchors and for TRON using the newer global syntax.

It sounds like the HTML rig introduces noticeable performance impact and we should look at that, but I don't think that affects straight Tangram use of the scene styles, or Mapzen.js use of the scene files. @bcamper can you provide further detail on the performance hit and how to replicate it, please?

I agree longer term we should move towards a more generic system than these dozens of globals, but right now these are internal variables that have been around for a while. Tracking general conditionals (globals) for label with #236.

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

No branches or pull requests

1 participant