cusip._alphabet is too restrictive: prevents matching valid CUSIPs

[thomkav]

The cusip module's is_valid function returns false for a CUSIP that I would expect would pass, as I found it on a real Fidelity statement (see screenshot of document sample).

cusip.is_valid(`FDIC99425`)
>> False

I believe this is because cusip._alphabet does not permit "I" or "O":

cusip._alphabet
>> '0123456789ABCDEFGH JKLMN PQRSTUVWXYZ*@#'

I checked cusip.com and I didn't find anything indicating that those two characters were not allowed.

[arthurdejong]

Thanks for pointing this out. It's been fixed in 1364e19

[thomkav]

A long awaited thanks @arthurdejong !