Bug in PresidioAnalyzerWrapper: 'ORGANIZATION' is not removed by default

[ebotiab]

The method _update_recognizers_based_on_entities_to_keep add "ORGANIZATION" entity:

presidio-research/presidio_evaluator/models/presidio_analyzer_wrapper.py

Lines 103 to 108 in d105ee2

	Add ORGANIZATION as it is removed by default
	"""
	supported_entities = analyzer_engine.get_supported_entities(
	language=self.language
	)

presidio-research/presidio_evaluator/models/presidio_analyzer_wrapper.py

Line 121 in d105ee2

if "ORGANIZATION" in self.entities and "ORGANIZATION" not in supported_entities:

But it is actually not removed by default, which could be checked just by running:

ORGANIZATION" in AnalyzerEngine().get_supported_entities("en")
True  # output

[omri374]

If I understand correctly, this depends on your Presidio analyzer version and configuration. This logic adds ORGANIZATION in case it was removed in the Presidio Analyzer logic. It could be that the Analyzer version actually has ORGANIZATION inside, therefore this logic is not needed.

Is that the case?