Default VirtualMachine to RandomBestLeastRecentlyViewed

Author: desplesda

CHANGELOG.md

@@ -11,6 +11,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 ### Updated
 
 - Complexity scoring for `when:` headers no longer counts `not` operations. This means that `when: $x` and `when: not $x` have the same score of 1.
+- `VirtualMachine.ContentSaliencyStrategy` now defaults to `RandomBestLeastRecentlyViewed`, not `First`
 
 ### Removed
 

YarnSpinner/VirtualMachine.cs

@@ -239,6 +239,7 @@ internal VirtualMachine(Library library, IVariableStorage storage)
         {
             this.Library = library;
             this.VariableStorage = storage;
+            this.ContentSaliencyStrategy = new RandomBestLeastRecentlyViewedSaliencyStrategy(storage);
             state = new State();
         }
 
@@ -323,7 +324,7 @@ private set
             }
         }
 
-        public IContentSaliencyStrategy ContentSaliencyStrategy { get; internal set; } = new FirstSaliencyStrategy();
+        public IContentSaliencyStrategy ContentSaliencyStrategy { get; internal set; }
 
         internal Node? currentNode;