Skip to content

refactor: simplify index retrieval in InstanceCounterAnonymizer#1577

Merged
omri374 merged 1 commit into
mainfrom
fix_psedonymization_get_index
May 7, 2025
Merged

refactor: simplify index retrieval in InstanceCounterAnonymizer#1577
omri374 merged 1 commit into
mainfrom
fix_psedonymization_get_index

Conversation

@RonShakutai

@RonShakutai RonShakutai commented May 7, 2025

Copy link
Copy Markdown
Collaborator

Change Description

Refactor the pseudonymization sample notebook to simplify how we compute the next index for a given entity type.
Removed the manual string-parsing logic in _get_last_index and replaced it with a direct return len(entity_mapping_for_type) call for clarity and performance.

Checklist

@RonShakutai RonShakutai self-assigned this May 7, 2025
@RonShakutai RonShakutai requested a review from omri374 May 7, 2025 15:18

@omri374 omri374 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@omri374 omri374 merged commit 452d6bb into main May 7, 2025
@omri374 omri374 deleted the fix_psedonymization_get_index branch May 7, 2025 20:01
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.

2 participants