-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Description
scikit-psl/skpsl/estimators/probabilistic_scoring_list.py
Lines 238 to 248 in ccd105c
len_pre = min(len(set(predef_features) & remaining_features), len_) | |
len_rest = len_ - len_pre | |
if strict and predef_features: | |
prefixes = [ | |
[f_ for f_ in predef_features if f_ in remaining_features][:len_pre] | |
] | |
else: | |
prefixes = permutations( | |
set(predef_features) & remaining_features, len_pre | |
) |
lookahead should not consider any additional features for selection untill the prefix is exausted.
maybe that is already the case, and maybe we need the lookahead regardless because we might need to lookahead when selecting the score for the current feature.
just needs to be checked, as it might save runtime
Metadata
Metadata
Assignees
Labels
No labels