Adding back some helpful comments

Author: ChSonnabend

GPU/GPUTracking/Base/cuda/GPUReconstructionCUDA.cu

@@ -655,14 +655,20 @@ void GPUReconstructionCUDA::SetONNXGPUStream(Ort::SessionOptions& sessionOptions
   // CUDA is the fallback for nodes unsupported by TensorRT.
   OrtCUDAProviderOptionsV2* cudaOptions = nullptr;
   ORTCHK(api->CreateCUDAProviderOptions(&cudaOptions));
+  // std::vector<const char*> keys{"device_id", "gpu_mem_limit", "arena_extend_strategy", "cudnn_conv_algo_search", "do_copy_in_default_stream", "cudnn_conv_use_max_workspace", "cudnn_conv1d_pad_to_nc1d"};
+  // std::vector<const char*> values{"0", "2147483648", "kSameAsRequested", "DEFAULT", "1", "1", "1"};
+  // UpdateCUDAProviderOptions(cuda_options, keys.data(), values.data(), keys.size());
   ORTCHK(api->UpdateCUDAProviderOptionsWithValue(cudaOptions, "user_compute_stream", mInternals->Streams[stream]));
   ORTCHK(api->SessionOptionsAppendExecutionProvider_CUDA_V2(sessionOptions, cudaOptions));
   api->ReleaseCUDAProviderOptions(cudaOptions);
 
 #elif defined(ORT_ROCM_BUILD)
+  // const auto& api = Ort::GetApi();
+  // api.GetCurrentGpuDeviceId(deviceId);
   OrtROCMProviderOptions rocmOptions;
-  rocmOptions.has_user_compute_stream = 1;
-  rocmOptions.arena_extend_strategy = 0;
+  rocmOptions.has_user_compute_stream = 1; // Indicate that we are passing a user stream
+  rocmOptions.arena_extend_strategy = 0; // kNextPowerOfTwo = 0, kSameAsRequested = 1 -> https://github.com/search?q=repo%3Amicrosoft%2Fonnxruntime%20kSameAsRequested&type=code
+  // rocm_options.gpu_mem_limit = 1073741824; // 0 means no limit
   rocmOptions.user_compute_stream = mInternals->Streams[stream];
   sessionOptions.AppendExecutionProvider_ROCM(rocmOptions);
 #endif