Skip to content

Added the option to add custom operators + pseudonymization sample#1284

Merged
omri374 merged 13 commits into
mainfrom
omri/add_custom_operators
Feb 12, 2024
Merged

Added the option to add custom operators + pseudonymization sample#1284
omri374 merged 13 commits into
mainfrom
omri/add_custom_operators

Conversation

@omri374

@omri374 omri374 commented Feb 9, 2024

Copy link
Copy Markdown
Collaborator

Change Description

Added add and remove methods for operators, allowing users to create custom operators (inheriting from the Operator class).

This required some refactoring to the OperatorFactory class.

Documentation to follow in a future PR, notebook can be found here: https://github.com/microsoft/presidio/blob/omri/add_custom_operators/docs/samples/python/pseudonomyzation.ipynb

Issue reference

#1052, #1093, #1246,

Checklist

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

Comment thread presidio-anonymizer/tests/conftest.py Outdated
@omri374 omri374 requested a review from roeybc February 10, 2024 21:33
@omri374 omri374 changed the title Added the option to add custom operators Added the option to add custom operators + pseudonymization sample Feb 10, 2024
Comment thread docs/samples/python/pseudonomyzation.ipynb
Comment thread presidio-anonymizer/presidio_anonymizer/anonymizer_engine.py Outdated
Comment thread presidio-anonymizer/presidio_anonymizer/anonymizer_engine.py Outdated
Comment thread presidio-anonymizer/presidio_anonymizer/deanonymize_engine.py
Comment thread presidio-anonymizer/presidio_anonymizer/anonymizer_engine.py
shiranr
shiranr previously approved these changes Feb 12, 2024

@shiranr shiranr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me :) Left couple of short remarks.

Co-authored-by: Shiran Rubin <shiranrubin@microsoft.com>
@omri374 omri374 merged commit ee84d70 into main Feb 12, 2024
@omri374 omri374 deleted the omri/add_custom_operators branch February 12, 2024 13:45
prokopidis pushed a commit to prokopidis/presidio that referenced this pull request Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants