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

Unable to have multiple CesiumWidets on the same page in 3d-tiles branch #5226

Closed
hpinkos opened this issue Apr 21, 2017 · 5 comments
Closed

Comments

@hpinkos
Copy link
Contributor

hpinkos commented Apr 21, 2017

In this example, I'm creating a new CesiumWidget for each of 4 containers. However, the globe doesn't render in two of them. This example works in master.

<style>
    @import url(../templates/bucket.css);
</style>
<table id="cesiumTable"></table>
<div id="loadingOverlay"><h1>Loading...</h1></div>
<div id="toolbar"></div>
var widgets = [];
var rows = 2;
var cols = 2;
var table = document.getElementById('cesiumTable');
table.innerHTML = '';
for (var i = 0; i < rows; i++) {
    var tr = document.createElement('tr');
    table.appendChild(tr);
    for (var j = 0; j < cols; j++) {
        var td = document.createElement('td');
        tr.appendChild(td);
        var div = document.createElement('div');
        div.style.height = '200px';
        div.style.width = '400px';
        td.appendChild(div);
        widgets.push(new Cesium.CesiumWidget(div));
    }
}
@lilleyse
Copy link
Contributor

This is a bug related to the RequestScheduler. The scheduler budgets are stored at global scope but the budgets are reset in Scene, which would get called four times and mess up the budgets. If there is a good way to call RequestScheduler.resetBudgets just once per frame outside of any cesium widgets, that would be the fix.

@lilleyse
Copy link
Contributor

Also the request scheduler architecture has potential to change with @austinEng's HTTP2 testing.

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 22, 2017

This should be fixed before 3d-tiles is merged into master.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jun 14, 2017

Already fixed in #5308?

@mramato
Copy link
Contributor

mramato commented Jun 14, 2017

Yes

@mramato mramato closed this as completed Jun 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants