General Update: [FIX] ensure a default value is specified for optional parameters defined via interface

Author: Cryptoc1

LLama/LLamaInstructExecutor.cs

@@ -101,7 +101,7 @@ public override async Task SaveState(string filename, CancellationToken cancella
             }
         }
         /// <inheritdoc />
-        public override async Task LoadState(string filename, CancellationToken cancellationToken)
+        public override async Task LoadState(string filename, CancellationToken cancellationToken = default)
         {
             using (var fs = new FileStream(filename, FileMode.Open, FileAccess.Read))
             {

LLama/LLamaInteractExecutor.cs

@@ -69,7 +69,7 @@ public override ExecutorBaseState GetStateData()
             return state;
         }
         /// <inheritdoc />
-        public override Task LoadState(ExecutorBaseState data, CancellationToken cancellationToken)
+        public override Task LoadState(ExecutorBaseState data, CancellationToken cancellationToken = default)
         {
             if (data is InteractiveExecutorState state)
             {
@@ -203,6 +203,7 @@ private Task PreprocessLlava(string text, InferStateArgs args, bool addBos = tru
         /// </summary>
         /// <param name="inferenceParams"></param>
         /// <param name="args"></param>
+        /// <param name="cancellationToken"></param>
         /// <returns></returns>
         protected override Task<(bool, IReadOnlyList<string>)> PostProcess(IInferenceParams inferenceParams, InferStateArgs args, CancellationToken cancellationToken)
         {