-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error while detecting US SSN and US Bank account number. #34
Comments
This also happens for US_Driver_licence. After modifying all these 3 in the csv file, the error goes away. |
This is happening for AU_MEDICARE entity type as well. |
Thanks for bringing that up. The common_types are referenced in the CSV, so in doing the lookup for the common type, the CSV reference is found, although as stated not all types are supported. The mapping of types is being changed in a future release. |
@HardKothari I assume you modified this : https://github.com/EdyVision/pii-codex/blob/main/pii_codex/data/v1/pii_type_mappings.csv and how did you reload? (I'm using the notebook) GP |
I am using this locally on my pc and hence I just replaced the file in the library folder of my virtual environment. I am not sure how would it work with notebook 😞 |
It seems that when US_SSN is detected in the sentence it always errors to the below error message:
"Exception: An error occurred while processing the detected entity US_SSN"
Traceback:
After looking closer in the code it seems that this entity type is missing from the csv attached in the data folder.
file: pii_mapping_util.py
file: file_util.py
The file contains PII_Type = "US_SOCIAL_SECURITY_NUMBER" instead of "US_SSN"
Same exception happens for bank number as well:
Exception: An error occurred while processing the detected entity US_BANK_NUMBER
The file contains PII_Type = "US_BANK_ACCOUNT_NUMBER" instead of "US_BANK_NUMBER".
Hope this helps.
Thank you
The text was updated successfully, but these errors were encountered: