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

Ability to update Tileset options without reloading tileset #412

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

whitebozo
Copy link

@whitebozo whitebozo commented Feb 15, 2024

Added the ability to update tileset options without the need to reload full tileset.

@kring
Copy link
Member

kring commented Feb 16, 2024

Thanks for the PR @whitebozo! Can I bug you to sign our Contributor License Agreement so that we can review this?
https://github.com/CesiumGS/cesium/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

@whitebozo
Copy link
Author

Thanks for the PR @whitebozo! Can I bug you to sign our Contributor License Agreement so that we can review this? https://github.com/CesiumGS/cesium/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

No problem I sent over the details to my boss waiting for him to fill it out on behalf of our company.

I will reply in here once we have signed the CLA.

@j9liu
Copy link
Contributor

j9liu commented Mar 21, 2024

Hi @whitebozo, did you end up signing a CLA? Let us know if this is ready for a review, we'll be able to take a look!

@whitebozo
Copy link
Author

Hi @whitebozo, did you end up signing a CLA? Let us know if this is ready for a review, we'll be able to take a look!

Still waiting for upper management to sign the CLA on behalf of our company. Sorry for the delay.

@electrum-bowie
Copy link

@kring

How LONG does it take you guys to pull this absolutely essential feature ??

I've been waiting for MONTHS thinking it will be added in the next version.....

EVERY Project that allows some kind of LOD / Performance option is absolutely ruined by the outrageous "reload" effect, EVERY. SINGLE. TIME..

It COMPLETELY breaks the entire game / app experience, and MOST apps / games with Cesium needs some kind of Level of Detail slider ...

Funny how nobody cares assuming it isn't possible to implement when EVERYTHING is already developed and completely implemented ready to solve this monstruos problem but instead it's sitting here for months ignored.

@whitebozo
Copy link
Author

whitebozo commented Jul 23, 2024

@kring

How LONG does it take you guys to pull this absolutely essential feature ??

I've been waiting for MONTHS thinking it will be added in the next version.....

EVERY Project that allows some kind of LOD / Performance option is absolutely ruined by the outrageous "reload" effect, EVERY. SINGLE. TIME..

It COMPLETELY breaks the entire game / app experience, and MOST apps / games with Cesium needs some kind of Level of Detail slider ...

Funny how nobody cares assuming it isn't possible to implement when EVERYTHING is already developed and completely implemented ready to solve this monstruos problem but instead it's sitting here for months ignored.

This is not the cesium team’s fault on the time it has taken for this feature to be pulled in my CEO never sent over a signed CLA so they can’t really pull this in without that. Feel free to take my PR and make another PR with a signed CLA. Until then or until my company signs the CLA the cesium team’s hands are tied. I’m sorry about the delay. The feature does work and we have been using internally at my company.

In hindsight I should have made this feature myself outside of my company’s umbrella and signed the CLA as an individual but since this PR is coming from my company’s GitHub account I had to go through the proper bureaucracy of getting the company to sign the CLA.

@kring
Copy link
Member

kring commented Jul 23, 2024

Thanks for opening this PR @whitebozo, and for continuing to push your management to bless the community's use of it. One thing that may help to persuade them is to point out that these changes likely have very little commercial value for you, and yet you're stuck re-integrating them into each new release. So it's actually to your company's benefit to sign the CLA and allow these changes to be merged, because it will save you work. It's not just a gift to the community.

Unfortunately, if someone else were to copy these changes and open a separate PR, we still wouldn't be able to merge it. The CLA requires that the contributor have the legal right to contribute the changes, and they wouldn't in this case because your company hasn't granted it. Someone else could re-implement this feature themselves, of course (including us at Cesium).

@electrum-bowie I understand your frustration, but please try to tone it down a little. Keep in mind that you always have the option of making your own changes to Cesium for Unity - or hiring someone to do so - when the product as-is isn't quite meeting your needs.

@raggnic
Copy link

raggnic commented Aug 12, 2024

Still waiting for upper management to sign the CLA on behalf of our company. Sorry for the delay.

I had the same issue. Management guys are very cautious in big companies

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.

5 participants