Fix: Classifier special case when no items are set to automatic matching #3858
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed change
Fixes a pretty special case, where items once existed to train the models with, but no items do any longer. Basically, if the file existed, the classifier is loaded by the consumer, then sent via signals to
set_xyz
, which uses it to predict items it shouldn't be any longer.As extra precaution, the prediction matching also ensures the item is still set to MATCH_AUTO. It's another special case where the matching item could have been changed from AUTO to something else, but the classifier hasn't yet been trained with that change.
Fixes #3848
Type of change
Checklist:
pre-commit
hooks, see documentation.