cesium-o3de icon indicating copy to clipboard operation
cesium-o3de copied to clipboard

O3DE 22.10 Shader fixes

Open lgleim opened this issue 1 year ago • 13 comments

Workaround fixes for shader compilation for O3DE 22.10 / current development branch Extends upon https://github.com/CesiumGS/cesium-o3de/pull/45 Fixes https://github.com/CesiumGS/cesium-o3de/issues/47

lgleim avatar Nov 10 '22 08:11 lgleim

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Dec 11 '22 03:12 cesium-concierge

The Cesium Gem uses the material->SetPropertyValue(...) function in multiple places within https://github.com/lgleim/cesium-o3de/blob/main/Code/Source/Cesium/TilesetUtility/GltfRasterMaterialBuilder.cpp Since the recent O3DE refactoring in https://github.com/o3de/o3de/commit/8d040facca379ad06ada1fcbeb09ddbcf9cdb19c the code fails to link with the following error thrown:

mold: error: undefined symbol: bool AZ::RPI::Material::SetPropertyValue<AZ::Data::Asset<AZ::RPI::ImageAsset> >(AZ::RHI::Handle<unsigned int, AZ::RPI::MaterialPropertyIndexType>, AZ::Data::Asset<AZ::RPI::ImageAsset> const&)
>>> referenced by GltfRasterMaterialBuilder.cpp
>>>               lib/profile/libCesium.Static.a(GltfRasterMaterialBuilder.cpp.o):(Cesium::GltfRasterMaterialBuilder::UnsetRasterForMaterial(unsigned int, AZStd::intrusive_ptr<AZ::RPI::Material>&))

Before the commit, the corresponding function could be found here: https://github.com/o3de/o3de/blob/ddbb3da2eff4e741fa355dd9e6d0475deee3ef44/Gems/Atom/RPI/Code/Source/RPI.Public/Material/Material.cpp#L603 @santorac Since you have a much better overview of the ongoing refactoring work, could you advise on how best to resolve this issue? I will then address this issue in my PR.

lgleim avatar Dec 16 '22 14:12 lgleim

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jan 16 '23 03:01 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Feb 15 '23 03:02 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Mar 17 '23 02:03 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Apr 16 '23 02:04 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar May 16 '23 02:05 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jun 15 '23 02:06 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jul 15 '23 02:07 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Aug 14 '23 02:08 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Sep 13 '23 02:09 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Oct 13 '23 02:10 cesium-concierge

Thanks again for your contribution @lgleim!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Nov 12 '23 03:11 cesium-concierge