Merge pull request #168 from advieser/fix/hash

fix:: calculating hash
mlr-org · Sep 24, 2024 · a43d4e3 · a43d4e3
2 parents da037e1 + 19c5b2d
commit a43d4e3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -7,6 +7,7 @@
 - Added `FilterUnivariateCox` (thanks to @bblodfon)
 - Parameter value `na.rm` is properly initialized to `TRUE` (thanks to @bblodfon)
 - Bugfix: property `missings` is now set correctly for `FilterFindCorrelation`
+- Bugfix: `$hash` now works for `Filter`s
 
 # mlr3filters 0.7.1
 

diff --git a/R/Filter.R b/R/Filter.R
@@ -229,6 +229,10 @@ Filter = R6Class("Filter",
       assert_ro_binding(rhs)
       calculate_hash(class(self), self$id, mget(private$.extra_hash, envir = self))
     }
+  ),
+
+  private = list(
+    .extra_hash = character()
   )
 )