Fixed bug where bad scheduling could produce inconsistent CurrentValues. Fixes #108

Wouterdek · Wouterdek · commit b14a99ad4cad · 2021-05-02T15:20:19.000+02:00
diff --git a/NodeNetworkToolkit/ValueNode/ValueNodeOutputViewModel.cs b/NodeNetworkToolkit/ValueNode/ValueNodeOutputViewModel.cs
@@ -1,4 +1,5 @@
 ﻿using System;
+using System.Reactive.Concurrency;
 using NodeNetwork.ViewModels;
 using NodeNetwork.Views;
 using ReactiveUI;
@@ -38,7 +39,7 @@ public IObservable<T> Value
 
         public ValueNodeOutputViewModel()
         {
-            this.WhenAnyObservable(vm => vm.Value).ToProperty(this, vm => vm.CurrentValue, out _currentValue);
+            this.WhenAnyObservable(vm => vm.Value).ToProperty(this, vm => vm.CurrentValue, out _currentValue, false, Scheduler.Immediate);
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`using System;`
	`2`	`+using System.Reactive.Concurrency;`
`2`	`3`	`using NodeNetwork.ViewModels;`
`3`	`4`	`using NodeNetwork.Views;`
`4`	`5`	`using ReactiveUI;`
`@@ -38,7 +39,7 @@ public IObservable<T> Value`
`38`	`39`
`39`	`40`	`public ValueNodeOutputViewModel()`
`40`	`41`	`{`
`41`		`- this.WhenAnyObservable(vm => vm.Value).ToProperty(this, vm => vm.CurrentValue, out _currentValue);`
	`42`	`+ this.WhenAnyObservable(vm => vm.Value).ToProperty(this, vm => vm.CurrentValue, out _currentValue, false, Scheduler.Immediate);`
`42`	`43`	`}`
`43`	`44`	`}`
`44`	`45`	`}`