cleanco('AMBA').clean_name() is empty

[dbradshaw]

No description provided.

[petri]

What should it be, and why?

[dbradshaw]

I don't know. Maybe leave it as AMBA. Not familiar with the company.

…

On Wed, Sep 30, 2020, 4:33 PM Petri Savolainen ***@***.***> wrote: What should it be? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#57 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAQ2SHXTSGQ6J7UGWBHKLIDSIOI3JANCNFSM4PL3GYDQ> .

[petri]

We cannot fix this if we don't know what the result should be. Would you at least know in which country this "AMBA" is in?

[dbradshaw]

No. I came across it using cleanco and it caused a problem which I had to work around. I don't think I can help you further. Thanks for looking into it.

…

-David

On Sat, Oct 3, 2020 at 10:01 AM Petri Savolainen ***@***.***> wrote: We cannot fix this if we don't know what the result should be. Would you at least know in which country this "AMBA" is in? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#57 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAQ2SHTCYIGDEFAJGEZ2A73SI4VEFANCNFSM4PL3GYDQ> .

[FBnil]

This will always be a problem, a company that is named after a termdata (in this case from Denmark). terms_by_country might need to be limited to the countries the user expects/does business with or, the code needs to be changed that when you remove the last term, and the result is empty, rather than empty, you return the first word in the received string, as a "most probable name". To not break backwards compatibility, a new function would need to be made (or a flag might be set to change the behavior).

print(basename('AMBA')) # ""
print(basenameorfirst('AMBA')) # "AMBA"
print(basename('inc & co')) # ""
print(basenameorfirst('inc & co')) # "inc"

This would allow for companies name like 'inc & co', where even though inc is a term, it's also the name.