Render Graph - Fixing auto upgrade of moved namespace

Fixing missing moved from to make the API be auto updated.

https://jira.unity3d.com/browse/UUM-61578

Author: alex-vazquez-unity3d

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/IRenderGraphBuilder.cs

@@ -1,12 +1,12 @@
 using System;
-using System.Collections.Generic;
-using UnityEngine.Rendering;
+using UnityEngine.Scripting.APIUpdating;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     /// <summary>
     /// Common base interface for the different render graph builders. These functions are supported on all builders.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public interface IBaseRenderGraphBuilder : IDisposable
     {
         /// <summary>
@@ -132,6 +132,7 @@ public interface IBaseRenderGraphBuilder : IDisposable
     /// A builder for a compute render pass
     /// <see cref="RenderGraph.AddComputePass"/>
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public interface IComputeRenderGraphBuilder : IBaseRenderGraphBuilder
     {
 
@@ -149,6 +150,7 @@ public void SetRenderFunc<PassData>(BaseRenderFunc<PassData, ComputeGraphContext
     /// A builder for an unsafe render pass.
     /// <see cref="RenderGraph.AddUnsafePass"/>
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public interface IUnsafeRenderGraphBuilder : IBaseRenderGraphBuilder
     {
         /// <summary>
@@ -165,6 +167,7 @@ public void SetRenderFunc<PassData>(BaseRenderFunc<PassData, UnsafeGraphContext>
     /// A builder for a raster render pass
     /// <see cref="RenderGraph.AddRasterRenderPass"/>
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public interface IRasterRenderGraphBuilder : IBaseRenderGraphBuilder
     {
         /// <summary>

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/IRenderGraphRecorder.cs

@@ -1,8 +1,11 @@
+using UnityEngine.Scripting.APIUpdating;
+
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     /// <summary>
     /// A base class that can be inherited by render graph implementers. Currently this is only used by URP. In the future this will allow shared SRP rendering features between HDRP and URP through render graph.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public interface IRenderGraphRecorder
     {
         /// <summary>

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraph.Compiler.cs

@@ -1,20 +1,19 @@
-using System;
-using UnityEngine.Rendering;
+using UnityEngine.Rendering.RenderGraphModule.NativeRenderPassCompiler;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     public partial class RenderGraph
     {
         //TODO(ddebaets) move old compile func/members over
 
-        NativeRenderPassCompiler.NativePassCompiler nativeCompiler = null;
+        NativePassCompiler nativeCompiler = null;
 
-        internal NativeRenderPassCompiler.NativePassCompiler CompileNativeRenderGraph(int graphHash)
+        internal NativePassCompiler CompileNativeRenderGraph(int graphHash)
         {
             using (new ProfilingScope(m_RenderGraphContext.cmd, ProfilingSampler.Get(RenderGraphProfileId.CompileRenderGraph)))
             {
                 if (nativeCompiler == null)
-                    nativeCompiler = new NativeRenderPassCompiler.NativePassCompiler(m_CompilationCache);
+                    nativeCompiler = new NativePassCompiler(m_CompilationCache);
 
                 bool compilationIsCached = nativeCompiler.Initialize(m_Resources, m_RenderPasses, m_DebugParameters.disablePassCulling, name, m_EnableCompilationCaching, graphHash, m_ExecutionCount);
                 if (!compilationIsCached)

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraph.cs

@@ -3,6 +3,7 @@
 using System.Diagnostics;
 using System.Runtime.CompilerServices;
 using UnityEngine.Experimental.Rendering;
+using UnityEngine.Scripting.APIUpdating;
 // Typedef for the in-engine RendererList API (to avoid conflicts with the experimental version)
 using CoreRendererListDesc = UnityEngine.Rendering.RendererUtils.RendererListDesc;
 
@@ -11,7 +12,7 @@ namespace UnityEngine.Rendering.RenderGraphModule
     /// <summary>
     /// Sets the read and write access for the depth buffer.
     /// </summary>
-    [Flags]
+    [Flags][MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public enum DepthAccess
     {
         ///<summary>Read Access.</summary>
@@ -25,7 +26,7 @@ public enum DepthAccess
     /// <summary>
     /// Express the operations the rendergraph pass will do on a resource.
     /// </summary>
-    [Flags]
+    [Flags][MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public enum AccessFlags
     {
         ///<summary>The pass does not access the resource at all. Calling Use* functions with none has no effect.</summary>
@@ -51,6 +52,7 @@ public enum AccessFlags
     /// An object representing the internal context of a rendergraph pass execution.
     /// This object is public for technical reasons only and should not be used.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public class InternalRenderGraphContext
     {
         internal ScriptableRenderContext renderContext;
@@ -83,6 +85,7 @@ internal interface IDerivedRendergraphContext
     /// command buffer that can be used to schedule all commands. This will eventually be deprecated
     /// in favor of more specific contexts that have more specific command buffer types.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RenderGraphContext : IDerivedRendergraphContext
     {
         private InternalRenderGraphContext wrappedContext;
@@ -107,6 +110,7 @@ public void FromInternalContext(InternalRenderGraphContext context)
     /// This class declares the context object passed to the execute function of a raster render pass.
     /// <see cref="RenderGraph.AddRasterRenderPass"/>
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RasterGraphContext : IDerivedRendergraphContext
     {
         private InternalRenderGraphContext wrappedContext;
@@ -135,6 +139,7 @@ public void FromInternalContext(InternalRenderGraphContext context)
     /// This class declares the context object passed to the execute function of a compute render pass.
     /// <see cref="RenderGraph.AddComputePass"/>
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public class ComputeGraphContext : IDerivedRendergraphContext
     {
         private InternalRenderGraphContext wrappedContext;
@@ -164,6 +169,7 @@ public void FromInternalContext(InternalRenderGraphContext context)
     /// This class declares the context object passed to the execute function of an unsafe render pass.
     /// <see cref="RenderGraph.AddUnsafePass"/>
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public class UnsafeGraphContext : IDerivedRendergraphContext
     {
         private InternalRenderGraphContext wrappedContext;
@@ -191,6 +197,7 @@ public void FromInternalContext(InternalRenderGraphContext context)
     /// <summary>
     /// This struct contains properties which control the execution of the Render Graph.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RenderGraphParameters
     {
         ///<summary>Identifier for this render graph execution.</summary>
@@ -216,12 +223,14 @@ public struct RenderGraphParameters
     /// <typeparam name="ContextType">The type of the context that will be passed to the render function.</typeparam>
     /// <param name="data">Render Pass specific data.</param>
     /// <param name="renderGraphContext">Global Render Graph context.</param>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public delegate void BaseRenderFunc<PassData, ContextType>(PassData data, ContextType renderGraphContext) where PassData : class, new();
 
 
     /// <summary>
     /// This class is the main entry point of the Render Graph system.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public partial class RenderGraph
     {
         ///<summary>Maximum number of MRTs supported by Render Graph.</summary>
@@ -2148,7 +2157,7 @@ void PreRenderPassSetRenderTargets(in CompiledPassInfo passInfo, RenderGraphPass
                     else
                     {
                         throw new InvalidOperationException("Setting MRTs without a depth buffer is not supported.");
-                    } 
+                    }
                 }
                 else
                 {
@@ -2547,15 +2556,17 @@ internal TextureHandle GetGlobal(int globalPropertyId)
         }
     }
 
-#if DEVELOPMENT_BUILD || UNITY_EDITOR
     /// <summary>
     /// Render Graph Scoped Profiling markers
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RenderGraphProfilingScope : IDisposable
     {
+#if DEVELOPMENT_BUILD || UNITY_EDITOR
         ProfilingSampler m_Sampler;
         RenderGraph m_RenderGraph;
         bool m_Disposed;
+#endif
 
         /// <summary>
         /// Profiling Scope constructor
@@ -2564,10 +2575,12 @@ public struct RenderGraphProfilingScope : IDisposable
         /// <param name="sampler">Profiling Sampler to be used for this scope.</param>
         public RenderGraphProfilingScope(RenderGraph renderGraph, ProfilingSampler sampler)
         {
+#if DEVELOPMENT_BUILD || UNITY_EDITOR
             m_RenderGraph = renderGraph;
             m_Sampler = sampler;
             m_Disposed = false;
             renderGraph.BeginProfilingSampler(sampler);
+#endif
         }
 
         /// <summary>
@@ -2581,6 +2594,7 @@ public void Dispose()
         // Protected implementation of Dispose pattern.
         void Dispose(bool disposing)
         {
+#if DEVELOPMENT_BUILD || UNITY_EDITOR
             if (m_Disposed)
                 return;
 
@@ -2593,34 +2607,7 @@ void Dispose(bool disposing)
             }
 
             m_Disposed = true;
+#endif
         }
     }
-#else
-    /// <summary>
-    /// Render Graph Scoped Profiling markers
-    /// </summary>
-    public struct RenderGraphProfilingScope : IDisposable
-    {
-        /// <summary>
-        /// Profiling Scope constructor
-        /// </summary>
-        /// <param name="renderGraph">Render Graph used for this scope.</param>
-        /// <param name="sampler">Profiling Sampler to be used for this scope.</param>
-        public RenderGraphProfilingScope(RenderGraph renderGraph, ProfilingSampler sampler)
-        {
-        }
-
-        /// <summary>
-        ///  Dispose pattern implementation
-        /// </summary>
-        public void Dispose()
-        {
-        }
-
-        // Protected implementation of Dispose pattern.
-        void Dispose(bool disposing)
-        {
-        }
-    }
-#endif
 }

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphBuilder.cs

@@ -1,13 +1,13 @@
 using System;
 using System.Diagnostics;
-using System.Collections.Generic;
-using UnityEngine.Rendering;
+using UnityEngine.Scripting.APIUpdating;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     /// <summary>
     /// Use this struct to set up a new Render Pass.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RenderGraphBuilder : IDisposable
     {
         RenderGraphPass m_RenderPass;

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphBuilders.cs

@@ -1,10 +1,6 @@
 using System;
 using System.Diagnostics;
-using System.Collections.Generic;
 using UnityEngine.Experimental.Rendering;
-using UnityEngine.Rendering;
-using static UnityEngine.Rendering.DebugUI;
-using ValueTuple = System.ValueTuple;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
@@ -232,7 +228,7 @@ private ResourceHandle UseResource(in ResourceHandle handle, AccessFlags flags)
                 ValidateWriteTo(handle);
                 m_RenderPass.AddResourceWrite(m_Resources.GetNewVersionedHandle(handle));
                 m_Resources.IncrementWriteCount(handle);
-            }  
+            }
 
             return GetLatestVersionHandle(handle);
         }

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphDefaultResources.cs

@@ -1,10 +1,11 @@
-using UnityEngine.Rendering;
+using UnityEngine.Scripting.APIUpdating;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     /// <summary>
     /// Helper class allowing access to default resources (black or white texture, etc.) during render passes.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public class RenderGraphDefaultResources
     {
         // We need to keep around a RTHandle version of default regular 2D textures since RenderGraph API is all RTHandle.

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphResourceAccelerationStructure.cs

@@ -1,14 +1,13 @@
-using System;
 using System.Diagnostics;
-
-using UnityEngine.Rendering;
+using UnityEngine.Scripting.APIUpdating;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     /// <summary>
     /// RayTracingAccelerationStructure resource handle.
     /// </summary>
     [DebuggerDisplay("RayTracingAccelerationStructure ({handle.index})")]
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RayTracingAccelerationStructureHandle
     {
         private static RayTracingAccelerationStructureHandle s_NullHandle = new RayTracingAccelerationStructureHandle();
@@ -41,6 +40,7 @@ public struct RayTracingAccelerationStructureHandle
     /// <summary>
     /// Descriptor used to identify ray tracing acceleration structure resources
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RayTracingAccelerationStructureDesc
     {
         /// <summary>RayTracingAccelerationStructure name.</summary>

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphResourceBuffer.cs

@@ -1,12 +1,13 @@
-using System;
 using System.Diagnostics;
+using UnityEngine.Scripting.APIUpdating;
 
 namespace UnityEngine.Rendering.RenderGraphModule
 {
     /// <summary>
     /// Graphics Buffer resource handle.
     /// </summary>
     [DebuggerDisplay("Buffer ({handle.index})")]
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct BufferHandle
     {
         // Minor Warning: This calls the zeroing constructor this means that the embedded ResourceHandle struct will also be zero-ed

Packages/com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphResourceRegistry.cs

@@ -1,11 +1,9 @@
 using System;
-using System.Diagnostics;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Runtime.CompilerServices;
 using UnityEngine.Experimental.Rendering;
-using UnityEngine.Rendering;
-using UnityEngine.Rendering.RendererUtils;
-
+using UnityEngine.Scripting.APIUpdating;
 // Typedefs for the in-engine RendererList API (to avoid conflicts with the experimental version)
 using CoreRendererList = UnityEngine.Rendering.RendererList;
 using CoreRendererListDesc = UnityEngine.Rendering.RendererUtils.RendererListDesc;
@@ -15,11 +13,12 @@ namespace UnityEngine.Rendering.RenderGraphModule
     /// <summary>
     /// Basic properties of a RTHandle needed by the render graph compiler. It is not always possible to derive these
     /// given an RTHandle to the user needs to pass these in.
-    /// 
+    ///
     /// We don't use a full RenderTargetDescriptor here as filling out a full descriptor may not be trivial and not all
     /// members of the descriptor are actually used by the render graph. This struct is the minimum set of info needed by the render graph.
     /// If you want to develop some utility framework to work with render textures, etc. it's probably better to use RenderTargetDescriptor.
     /// </summary>
+    [MovedFrom(true, "UnityEngine.Experimental.Rendering.RenderGraphModule", "UnityEngine.Rendering.RenderGraphModule")]
     public struct RenderTargetInfo
     {
         /// <summary>
@@ -516,7 +515,7 @@ internal TextureHandle ImportTexture(in RTHandle rt, in ImportResourceParams imp
             // (The alternative is for the code calling ImportTexture to use the overload that takes a RenderTargetInfo).
             if(rt != null)
                 ValidateRenderTarget(texHandle.handle);
-            
+
             return texHandle;
         }
 
@@ -675,7 +674,7 @@ internal TextureHandle ImportBackbuffer(RenderTargetIdentifier rt, in RenderTarg
         private void ValidateRenderTarget(in ResourceHandle res)
         {
             if(RenderGraph.enableValidityChecks)
-            {                
+            {
                 RenderTargetInfo outInfo;
                 GetRenderTargetInfo(res, out outInfo);
             }