Add sky controller prefab for sun positioning#187
Conversation
|
looking forward for the implementation. is this branch merge into main or any tags now? |
|
It's not available in any pre-built binaries. If you want to use this, the only option is to build from this PR's branch yourself. It's based on a very old version of Cesium for Unity, though, so you'll probably want to merge in main. In short, it's likely to be a lot of work. Sorry we don't have a simple solution right now. |
I did try to build cesium from this PR branch and try to run in some cesium example branches, sadly not working. yeap, this looks some old codes. I might not have time for a more deep dive into this. Anyway, looking forward for this to develped and merge into main branch and get release. UE sky looks cool, looking forward for Unity. |
|
Hi, |
|
@asafMasa Alex isn't working on this anymore unfortunately, but we'd be very happy to see someone from the community pick it up and run with it! |
|
Hi @kring, I'd like to pick this up and get sun/sky/atmosphere (#173) across the line. I saw this PR has been sitting since early 2023 and is based on a pretty old version of Cesium for Unity, and that you mentioned you'd be happy for someone from the community to run with it. I've spent some time digging into the problem and I'd like to take it on. Rather than try to revive this branch as-is, I'm planning to rebase the good ideas here onto current main and rework it into a few smaller, reviewable pieces. I went through the existing code here, the CesiumSunSky actor in cesium-unreal, and the CesiumJS Clock example to make sure I understand what "correct" looks like before writing anything. Here's the plan I'm proposing: Phase 1 - Sun + URP sky
Phase 2 - Clock + UI
Phase 3 - HDRP
Phase 4 - Integration + polish
A couple of things I ran into that I'd like your input on before I start:
If this direction sounds good I'll start on Phase 1 and open it as its own PR so it's easy to review in isolation. Thanks! |
|
Hi @baruchInsert-tech, thanks for picking this up! Your plan sounds reasonable to me, though @j9liu and @azrogers may want to weigh in as well. Regarding cesium-native versus C#, the main advantage to putting it in cesium-native is that it allows it to be used across engines. For example, if the sun position were implemented in C# and we wanted to add it to 3D Tiles for Godot, we would need to reimplement it. Adding it to cesium-native makes it more widely reusable. So I would say that's not a strict requirement, but as long as the burden isn't too unreasonably high for you, I would definitely recommend that you do it. |
This implements a sun, sky, and atmosphere solution for Cesium for Unity per #173.
Includes the following components:
It's currently a work in progress. Remaining items: