remove Binding initialization from VideoEffect implementation

Author: linnealovespie

Samples/StyleTransfer/AppViewModel.cs

@@ -41,8 +41,9 @@ public AppViewModel()
         private LearningModel m_model = null;
         private LearningModelDeviceKind m_inferenceDeviceSelected = LearningModelDeviceKind.Default;
         private LearningModelSession m_session;
-        string _inputImageDescription;
-        string _outputImageDescription;
+        private LearningModelBinding m_binding;
+        string m_inputImageDescription;
+        string m_outputImageDescription;
         IMediaExtension videoEffect;
 
 
@@ -168,10 +169,10 @@ public async Task ChangeLiveStream()
                 VideoEffectDefinition videoEffectDefinition = new VideoEffectDefinition("StyleTransferEffectComponent.StyleTransferVideoEffect");
                 videoEffect = await _mediaCapture.AddVideoEffectAsync(videoEffectDefinition, MediaStreamType.VideoPreview);
                 videoEffect.SetProperties(new PropertySet() {
-                    { "Model", m_model},
-                    { "Session", m_session },
-                    { "InputImageDescription", _inputImageDescription },
-                    { "OutputImageDescription", _outputImageDescription } });
+                    { "Session", m_session},
+                    { "Binding", m_binding },
+                    { "InputImageDescription", m_inputImageDescription },
+                    { "OutputImageDescription", m_outputImageDescription } });
 
                 StartPreview();
             }
@@ -213,12 +214,13 @@ private async Task LoadModelAsync()
             // TODO: Pass in useGPU as well.
             m_inferenceDeviceSelected = LearningModelDeviceKind.Cpu;
             m_session = new LearningModelSession(m_model, new LearningModelDevice(m_inferenceDeviceSelected));
+            m_binding = new LearningModelBinding(m_session);
 
             debugModelIO();
 
-            _inputImageDescription = m_model.InputFeatures.ToList().First().Name;
+            m_inputImageDescription = m_model.InputFeatures.ToList().First().Name;
 
-            _outputImageDescription = m_model.OutputFeatures.ToList().First().Name;
+            m_outputImageDescription = m_model.OutputFeatures.ToList().First().Name;
         }
 
         public void debugModelIO()

Samples/StyleTransfer/StyleTransferEffectComponent/StyleTransferVideoEffect.cs

@@ -18,7 +18,6 @@ namespace StyleTransferEffectComponent
     public sealed class StyleTransferVideoEffect : IBasicVideoEffect
     {
 
-        private LearningModel _model = null;
         private LearningModelSession _session;
         private LearningModelBinding _binding;
         string _inputImageDescription;
@@ -27,19 +26,14 @@ public sealed class StyleTransferVideoEffect : IBasicVideoEffect
         public void Close(MediaEffectClosedReason reason)
         {
             // Dispose of effect resources
-            if (_model != null) { _model = null; }
             if (_session != null) { _session = null; }
             if (_binding != null) { _binding = null; }
             if (_inputImageDescription != null) { _inputImageDescription = null; }
             if (_outputImageDescription != null) { _outputImageDescription = null; }
         }
 
 
-        private int frameCount;
-        public void DiscardQueuedFrames()
-        {
-            frameCount = 0;
-        }
+        public void DiscardQueuedFrames() { }
         public bool IsReadOnly { get { return false; } }
 
         private VideoEncodingProperties encodingProperties;
@@ -71,20 +65,20 @@ public void SetProperties(IPropertySet configuration)
             this.configuration = configuration;
         }
 
-
-        public LearningModel Model
+        public LearningModelBinding Binding
         {
             get
             {   // TODO: Read in model from configuration
                 // If null, then fail. 
                 object val;
-                if (configuration != null && configuration.TryGetValue("Model", out val))
+                if (configuration != null && configuration.TryGetValue("Binding", out val))
                 {
-                    return (LearningModel)val;
+                    return (LearningModelBinding)val;
                 }
                 return null; // Different default value/ raise exception
             }
         }
+
         public LearningModelSession Session
         {
             get
@@ -128,18 +122,16 @@ public string OutputImageDescription
         }
 
 
-
         public void ProcessFrame(ProcessVideoFrameContext context)
         {
             using (VideoFrame inputVideoFrame = context.InputFrame)
             using (VideoFrame outputVideoFrame = context.OutputFrame)
             {
-                _model = Model; // Don't even need to pass model, just session
+                _binding = Binding;
                 _session = Session;
                 _inputImageDescription = InputImageDescription;
                 _outputImageDescription = OutputImageDescription;
 
-                _binding = new LearningModelBinding(_session);
                 _binding.Bind(_inputImageDescription, ImageFeatureValue.CreateFromVideoFrame(inputVideoFrame));
                 _binding.Bind(_outputImageDescription, ImageFeatureValue.CreateFromVideoFrame(outputVideoFrame)); // Check if this correctly sets the context output