presidio icon indicating copy to clipboard operation
presidio copied to clipboard

changing predefined recognizers to use the config file

Open roeybc opened this issue 1 year ago • 1 comments

Change Description

This PR is going to accomplish the following:

  1. Refactor load_predefined_recognizers to load the settings from the default yml file
  2. Add warning in case recognizer is listed with a language not supported by the registry
  3. raise an exception in case there's discrepancy between the supported language in the registry and the one listed in the analyzer
  4. Edit the docs accordingly I guess :D

Checklist

  • [x] I have reviewed the contribution guidelines
  • [x] I have signed the CLA (if required)
  • [x] My code includes unit tests
  • [x] All unit tests and lint checks pass locally
  • [x] My PR contains documentation updates / additions if required

roeybc avatar May 30 '24 19:05 roeybc

image when refactored the codebase, added documentation & testing and you've got less text, you know it's a good change :D

roeybc avatar May 31 '24 02:05 roeybc

Currently I can't import from presidio_analyzer import RecognizerRegistryProvider, AnalyzerEngineProvider in presidio-analyzer 2.2.354. Will this release also solve that problem?

viclang avatar Jul 17 '24 09:07 viclang