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

Autoshare webgl renderers. #975

Merged
merged 3 commits into from
Apr 3, 2019
Merged

Autoshare webgl renderers. #975

merged 3 commits into from
Apr 3, 2019

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Feb 14, 2019

Autosharing is now the default. Renderers can be manually shared, and webgl layers can be manually be moved to different renderer.

To manually move a layer to a different renderer, for instance, given two layers: var layer1 = map.createLayer('feature', {renderer: 'webgl', autoshareRenderer: false}); var layer2 = map.createLayer('feature', {renderer: layer1.renderer(), autoshareRenderer: false});, the second layer can be moved to a new renderer by doing: layer2.switchRenderer(geo.createRenderer('webgl', layer2), true);.

This requires an update to vgl to work.

@manthey manthey force-pushed the shared-context-visibility branch 2 times, most recently from 563a1ae to 4faef39 Compare February 28, 2019 19:35
Autosharing is now the default.  Renderers can be manually shared, and
webgl layers can be manually be moved to different renderer.

To manually move a layer to a different renderer, for instance, given
two layers: `var layer1 = map.createLayer('feature', {renderer: 'webgl',
autoshareRenderer: false});  var layer2 = map.createLayer('feature',
{renderer: layer1.renderer(), autoshareRenderer: false});`, the second
layer can be moved to a new renderer by doing:
`layer2.switchRenderer(geo.createRenderer('webgl', layer2), true);`.
@manthey manthey changed the base branch from shared-context-visibility to master March 12, 2019 14:41
@aashish24
Copy link
Member

@manthey I am still reviewing this branch.. mostly done just has one or two things that I need to clear my self.

@manthey
Copy link
Contributor Author

manthey commented Apr 3, 2019

Can you also review OpenGeoscience/vgl#106, as this depend on it?

@aashish24
Copy link
Member

done.

@manthey manthey merged commit 3b8a970 into master Apr 3, 2019
@manthey manthey deleted the auto-merge-contexts branch April 3, 2019 15:36
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

Successfully merging this pull request may close these issues.

2 participants