Skip to content
This repository has been archived by the owner on Jul 17, 2024. It is now read-only.

3d tiles #3938

Merged
merged 1 commit into from
Oct 10, 2017
Merged

3d tiles #3938

merged 1 commit into from
Oct 10, 2017

Conversation

oterral
Copy link
Contributor

@oterral oterral commented Aug 30, 2017

Add generic 3d tileset managment for future tilesets vegetation and swissnames
Fix #3940

Test

@oterral oterral closed this Aug 30, 2017
@oterral oterral reopened this Aug 30, 2017
@loicgasser
Copy link
Contributor

�[1A�[2KPhantomJS 2.1.1 (Linux 0.0.0) ga_urlutils_service gaUrlUtils #shorten() "after each" hook for "handle service error displaying a log an returning the initial url" FAILED
Deferred tasks to flush (1): {id: 6, time: 8000}
verifyNoPendingTasks@test/lib/angular-mocks.js:2157:60
test/specs/UrlUtilsService.spec.js:174:38

@oterral
Copy link
Contributor Author

oterral commented Sep 11, 2017

Could be great to have tese 2 improvments before merging:
CesiumGS/cesium#5770 and CesiumGS/cesium#5788

'20161217',
'20161121',
'20160909'
'20170425',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't forget to update the codepen and doc in chsdi.

// Url of olcesium library
var olCesiumLibUrl = gaGlobalOptions.resourceUrl;
console.log(olCesiumLibUrl);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

console

@loicgasser
Copy link
Contributor

@oterral FYI: both PRs you mentionned have been merged.

@oterral
Copy link
Contributor Author

oterral commented Sep 26, 2017

Yes but CesiumGS/cesium#5770 has introduce a bug. I need to check where.

@oterral oterral force-pushed the 3dtiles branch 3 times, most recently from bec85de to 091668b Compare September 28, 2017 15:21
@oterral
Copy link
Contributor Author

oterral commented Sep 28, 2017

Names added in layer manager in 3d:
Test

@davidoesch
Copy link
Contributor

Not in mobile

@davidoesch
Copy link
Contributor

Sorry I mean: hardly usable (with and without names) on ios11 iphone7 chrome. Serious lag compared to prod)

@loicgasser
Copy link
Contributor

On 3g samsung and chrome it s pretty good on my side on mobile. Samsung galaxy s8 rules

@davidoesch
Copy link
Contributor

I suspect ios11 .... has someone an ios10 available

@davidoesch
Copy link
Contributor

Confirmed by moc: Not functional on ios11 iphone7

@gjn
Copy link
Contributor

gjn commented Sep 29, 2017

I've alerted Tom.

@gjn
Copy link
Contributor

gjn commented Sep 29, 2017

@davidoesch Anyhow: is the name behaviour ok as it is? If yes, we can give it to Topo for Abnahme.

@gjn
Copy link
Contributor

gjn commented Sep 29, 2017

@cedricmoullet @davidoesch Can you update your ios11? It works fine on Tom's ios11.

@cedricmoullet
Copy link
Contributor

cedricmoullet commented Sep 29, 2017 via email

@davidoesch
Copy link
Contributor

Name behaviour is ok. I will organize acceptance and komm mmgt with Booth involved Departements

@oterral
Copy link
Contributor Author

oterral commented Oct 10, 2017

swissnames and trees has been removed from this PR.

@oterral oterral merged commit 64ceb24 into master Oct 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants