-
Notifications
You must be signed in to change notification settings - Fork 98
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
Base name is not working with some names #77
Comments
Yes, the point of basename is removing common suffixes, prefixes etc. to leave just the base name. You're basically giving those suffixes/prefixes there, or combinations of them. What is the problem you're having with this? Are those actual company names that you try to normalize? |
I think the code, in the last iteration removing things, if it finds that it has to remove everything, there must be a way to recover the iteration before that. (but maybe not by default, because it's actually handy to remove multiple terms). Of course, this check can be done at the userside too, and should at least be mentioned in the readme/documentation. |
Yes, agree with you |
Yes, these are the some of the organization names I came across. |
I checked some of the scenarios where basename function giving empty result.
from cleanco import basename
print("Base name name for {} : {}".format('IKS APS', basename("IKS APS")))
print("Base name name for {} : {}".format('S.C.S & COMPANY', basename("S.C.S & COMPANY")))
print("Base name name for {} : {}".format('COOP', basename("COOP")))
The text was updated successfully, but these errors were encountered: