Script canvas logger can now identify graph and entity names

[guillaume-haerinck]

What does this PR do?

Work on #9192, #2583, #17643, #10795. Script canvas logger can now show graph names and entity names. Code has been mostly restored to the Lumberyard's state.

GraphInfo class has been moved back to its old state so that it can be serialized (only used by the debugger to send remote messages, make no sense to hold a ptr to execution state there)

There are two things which still needs to be done to have all of the functionnalities back:

• Fix the focus on double click in log entry (broken by redcoding asset handler, holder, tracker, memory asset #6394 and for now I'm unsure if there is an alternative already in the engine to get a NodeId from its assetNodeId)
• Fill the entities tab with entities spawned and placed in the scene

inaction.mp4

How was this PR tested?

Locally against the newspaper sample project, with multiple graphs

[lsemp3d]

Excellent work, thank you for taking it on!

[guillaume-haerinck]

Some context for future ref about the focus issue :
In GraphSerialization.cpp, on graph open and graph save done via Deserialize(), all of the node Ids are randomized via MakeGraphComponentEntityIdsUnique(). If I disable this behavior and grab the graph I'm able to have a match between my debugger nodeIds and the runtime graph nodeIds, (and focus works after that).

Fix will be to track these remapped IDs and get them on demand, as it was possible before.

[spham-amzn]

Looks like a legit n AR failure:

[2024-05-12T15:26:47.430Z] /usr/bin/clang++-12 -DAZ_BUILD_CONFIGURATION_TYPE=\"profile\" -DAZ_ENABLE_DEBUG_TOOLS -DAZ_ENABLE_TRACING -DAZ_PROFILE_BUILD -DENABLE_EXTENDED_MATH_SUPPORT=0 -DLINUX -DLINUX64 -DNDEBUG -DSCRIPTCANVAS -DSCRIPTCANVAS_ERRORS_ENABLED -DSC_EXECUTION_TRACE_ENABLED -D_FORTIFY_SOURCE=2 -D_HAS_EXCEPTIONS=0 -D_PROFILE -D__linux__ -DCMAKE_INTDIR=\"profile\" -I/data/workspace/o3de/Gems/ScriptCanvas/Code/. -I/data/workspace/o3de/Gems/ScriptCanvas/Code/Include -I/data/workspace/o3de/Gems/ScriptCanvas/Code/Include/ScriptCanvas -I/data/workspace/o3de/build/linux/External/ScriptCanvas-8caffd05/Code/Azcg/Generated/ScriptCanvas.API -I/data/workspace/o3de/Code/Framework/AzCore/. -I/data/workspace/o3de/Code/Framework/AzCore/Platform/Linux -I/data/workspace/o3de/Code/Framework/AzCore/Platform/Common -I/data/workspace/o3de/Code/Framework/AzFramework/. -I/data/workspace/o3de/Code/Framework/AzFramework/Platform/Linux -I/data/workspace/o3de/Code/Framework/AzNetworking/. -I/data/workspace/o3de/Code/Framework/AzNetworking/Platform/Common -I/data/workspace/o3de/Code/Framework/AzNetworking/Platform/Linux -I/data/workspace/o3de/build/linux/Code/Framework/AzNetworking/Azcg/Generated/AzNetworking -I/data/workspace/o3de/Gems/ScriptEvents/Code/Include -I/data/workspace/o3de/Gems/ExpressionEvaluation/Code/Include -isystem /data/workspace/3rdParty/packages/Lua-5.4.4-rev1-linux/Lua/include -isystem /data/workspace/3rdParty/packages/RapidJSON-1.1.0-rev1-multiplatform/RapidJSON/include -isystem /data/workspace/3rdParty/packages/RapidXML-1.13-rev1-multiplatform/RapidXML/include -isystem /data/workspace/3rdParty/packages/zlib-1.2.11-rev5-linux/zlib/include -isystem /data/workspace/3rdParty/packages/cityhash-1.1-multiplatform/cityhash/src -isystem /data/workspace/3rdParty/packages/lz4-1.9.4-rev2-linux/lz4/include -fno-exceptions -fvisibility=hidden -fvisibility-inlines-hidden -Wall -Werror -Wno-inconsistent-missing-override -Wrange-loop-analysis -Wno-unknown-warning-option -Wno-parentheses -Wno-reorder -Wno-switch -Wno-undefined-var-template -msse4.1  -O2 -g -fstack-protector-all -fstack-check -fPIC -std=c++17 -MD -MT External/ScriptCanvas-8caffd05/Code/CMakeFiles/ScriptCanvas.API.dir/profile/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp.o -MF External/ScriptCanvas-8caffd05/Code/CMakeFiles/ScriptCanvas.API.dir/profile/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp.o.d -o External/ScriptCanvas-8caffd05/Code/CMakeFiles/ScriptCanvas.API.dir/profile/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp.o -c /data/workspace/o3de/Gems/ScriptCanvas/Code/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp

[2024-05-12T15:26:47.430Z] /data/workspace/o3de/Gems/ScriptCanvas/Code/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp:204:31: error: unused variable 'subgraphId' [-Werror,-Wunused-variable]

[2024-05-12T15:26:47.430Z]             AZ::Data::AssetId subgraphId = AZ::Data::AssetId::CreateString(assetIdString);

[2024-05-12T15:26:47.430Z]                               ^

[guillaume-haerinck]

Looks like a legit n AR failure:

[2024-05-12T15:26:47.430Z] /usr/bin/clang++-12 -DAZ_BUILD_CONFIGURATION_TYPE=\"profile\" -DAZ_ENABLE_DEBUG_TOOLS -DAZ_ENABLE_TRACING -DAZ_PROFILE_BUILD -DENABLE_EXTENDED_MATH_SUPPORT=0 -DLINUX -DLINUX64 -DNDEBUG -DSCRIPTCANVAS -DSCRIPTCANVAS_ERRORS_ENABLED -DSC_EXECUTION_TRACE_ENABLED -D_FORTIFY_SOURCE=2 -D_HAS_EXCEPTIONS=0 -D_PROFILE -D__linux__ -DCMAKE_INTDIR=\"profile\" -I/data/workspace/o3de/Gems/ScriptCanvas/Code/. -I/data/workspace/o3de/Gems/ScriptCanvas/Code/Include -I/data/workspace/o3de/Gems/ScriptCanvas/Code/Include/ScriptCanvas -I/data/workspace/o3de/build/linux/External/ScriptCanvas-8caffd05/Code/Azcg/Generated/ScriptCanvas.API -I/data/workspace/o3de/Code/Framework/AzCore/. -I/data/workspace/o3de/Code/Framework/AzCore/Platform/Linux -I/data/workspace/o3de/Code/Framework/AzCore/Platform/Common -I/data/workspace/o3de/Code/Framework/AzFramework/. -I/data/workspace/o3de/Code/Framework/AzFramework/Platform/Linux -I/data/workspace/o3de/Code/Framework/AzNetworking/. -I/data/workspace/o3de/Code/Framework/AzNetworking/Platform/Common -I/data/workspace/o3de/Code/Framework/AzNetworking/Platform/Linux -I/data/workspace/o3de/build/linux/Code/Framework/AzNetworking/Azcg/Generated/AzNetworking -I/data/workspace/o3de/Gems/ScriptEvents/Code/Include -I/data/workspace/o3de/Gems/ExpressionEvaluation/Code/Include -isystem /data/workspace/3rdParty/packages/Lua-5.4.4-rev1-linux/Lua/include -isystem /data/workspace/3rdParty/packages/RapidJSON-1.1.0-rev1-multiplatform/RapidJSON/include -isystem /data/workspace/3rdParty/packages/RapidXML-1.13-rev1-multiplatform/RapidXML/include -isystem /data/workspace/3rdParty/packages/zlib-1.2.11-rev5-linux/zlib/include -isystem /data/workspace/3rdParty/packages/cityhash-1.1-multiplatform/cityhash/src -isystem /data/workspace/3rdParty/packages/lz4-1.9.4-rev2-linux/lz4/include -fno-exceptions -fvisibility=hidden -fvisibility-inlines-hidden -Wall -Werror -Wno-inconsistent-missing-override -Wrange-loop-analysis -Wno-unknown-warning-option -Wno-parentheses -Wno-reorder -Wno-switch -Wno-undefined-var-template -msse4.1  -O2 -g -fstack-protector-all -fstack-check -fPIC -std=c++17 -MD -MT External/ScriptCanvas-8caffd05/Code/CMakeFiles/ScriptCanvas.API.dir/profile/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp.o -MF External/ScriptCanvas-8caffd05/Code/CMakeFiles/ScriptCanvas.API.dir/profile/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp.o.d -o External/ScriptCanvas-8caffd05/Code/CMakeFiles/ScriptCanvas.API.dir/profile/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp.o -c /data/workspace/o3de/Gems/ScriptCanvas/Code/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp

[2024-05-12T15:26:47.430Z] /data/workspace/o3de/Gems/ScriptCanvas/Code/Include/ScriptCanvas/Execution/Interpreted/ExecutionInterpretedDebugAPI.cpp:204:31: error: unused variable 'subgraphId' [-Werror,-Wunused-variable]

[2024-05-12T15:26:47.430Z]             AZ::Data::AssetId subgraphId = AZ::Data::AssetId::CreateString(assetIdString);

[2024-05-12T15:26:47.430Z]                               ^

That's weird the line is commented out, maybe you got the report from the old build

[guillaume-haerinck]

Yeah if you check the last report attached, it failed because of a timeout on linux build, but android passed

[guillaume-haerinck]

Now targeting stabilization branch