CesiumGS / cesium-unreal

Bringing the 3D geospatial ecosystem to Unreal Engine
https://cesium.com/platform/cesium-for-unreal/
Apache License 2.0
921 stars 295 forks source link

screenspacecontroller support for cesium unreal #1449

Closed jiangheng90 closed 2 months ago

jiangheng90 commented 3 months ago

fullfill the feature from community

jiangheng90 commented 3 months ago

the globe pawn is support the full feature when georeference origin placement in true origin,in cartographic some feature is disabled. Use the quick add panel. It will add in proper place.

FEJackFly commented 3 months ago

When is it online? It's urgent

chenguang1994 commented 3 months ago

expect

LuLuEarl commented 3 months ago

The operation is smooth,Thank you for sharing

jiangheng90 commented 3 months ago

@j9liu I have looked issue u link. so what do we need to do to make this move forward?

j9liu commented 3 months ago

Hey @jiangheng90, I did that while trying to organize PRs / issues in this repo. We have some things that we're trying to get done before the release next week, but we'll review this as soon as we can. Thank you for your patience :pray:

FEJackFly commented 2 months ago

image

What should I do when generating a solution and reporting error MSB307 is reported

ue 5.3 Visual Studio 2022 cmake 3.30

pjiang9 commented 2 months ago

@jiangheng90 Thank you for fulfilling the GlobePawn! Yet there are situations that will make Unreal Engine Crash. It is hard to describe how to reproduce the crash situations. It is likely to crash when zoom in and zoom out several times, and the viewport will be complete black, then UE will shut down. The following is the log captured when UE crashes. Much appreciated if you could look into it and fix it.

UE5.3,CesiumforUnreal2.5.0 UnrealEditor_CesiumRuntime!CesiumGeometry::Plane::Plane() UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::createCullingVolume() UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::ViewState::create() UnrealEditor_CesiumRuntime!ACesium3DTileset::CreateViewStateFromViewParameters() [E:\UnrealPack\CesiumforUE\UE_5.3\CesiumForUnreal\HostProject\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:1673] UnrealEditor_CesiumRuntime!ACesium3DTileset::Tick() [E:\UnrealPack\CesiumforUE\UE_5.3\CesiumForUnreal\HostProject\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:2123] UnrealEditor_Engine

UE5.3 CesiumforUnreal2.7.0 UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::createCullingVolume() UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::ViewState::create() UnrealEditor_CesiumRuntime!ACesium3DTileset::CreateViewStateFromViewParameters() [E:\UE53Projects\GlobePawn\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:1630] UnrealEditor_CesiumRuntime!ACesium3DTileset::Tick() [E:\UE53Projects\GlobePawn\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:2082]

andiven commented 2 months ago

@jiangheng90 I hope Touch support can be added.

kring commented 2 months ago

Hi @jiangheng90, sorry to see you close this. We've been slow to review it, but it's definitely a valuable contribution!

zzzsssddd999 commented 2 months ago

GlobePawn有BUG,用这个后,操作时UE编辑器运行窗口会突然黑屏,然后UE编辑器会崩溃(在编辑器运行后操作最多20分钟必出,该BUG概率挺高,最快可能1分钟就崩溃),UE5.3,在cesium unreal2.5,2.7版本都存在。换回cesium unreal自带的DynamicPawn再怎么操作都不会黑屏崩溃。

jiangheng90 commented 2 months ago

@kring I'm sorry for the pull request close.

It's been more than half a year since the end of development of this feature, and many of the technical details in it are not so familiar to me now.

Secondly, because of the development of this feature, some people found my social accounts and caused some trouble to my privacy, which I didn't want.

Right now, can u just make a feature branch and I make my pull request to this branch. it will take a long time to make this feature be all right and I think this is a better way for both of us to push it forward.

kring commented 1 month ago

It's been more than half a year since the end of development of this feature, and many of the technical details in it are not so familiar to me now.

Understood. It's only been a couple of months since you opened the PR, but even that is too long.

Secondly, because of the development of this feature, some people found my social accounts and caused some trouble to my privacy, which I didn't want.

Oh no, really sorry that's happening to you. That sort of behavior definitely has no place in the Cesium community. Let us know if we can help in any way.

Right now, can u just make a feature branch and I make my pull request to this branch. it will take a long time to make this feature be all right and I think this is a better way for both of us to push it forward.

Ok, I created a branch called screen-space-controller in this repo. It's just a copy of main.

jiangheng90 commented 1 month ago

@kring I have all ready make a new pull request.