nmapx / revolut-stocks-list

Extract Revolut stocks list from the list screenshot(s).
MIT License
74 stars 22 forks source link

[BUG] Wrong symbols #1

Closed lazyprogrammerio closed 4 years ago

lazyprogrammerio commented 4 years ago

In the list, you have wrong symbols:

BIL -> BILI GC -> GCI TIX -> TJX

Some of the stocks are dead/junk:

ARQL MDR PVTL ONCE SRCI STI

Thanks for the effort. If interested, I can give you the link to a public web app where you can check the stock details for the revolut ones.

nmapx commented 4 years ago

Indeed, there are some mistakes on the list due to the OCR inaccuracy. I'm still trying to figure out how to make it more precise to avoid it in the future. Regarding the dead stocks - it's also true but those are still on the list in Revolut app. I don't know if it's ok to remove those - someone might want to use this info for whatever reason 😄. We will see! It shouldn't be a big deal anyway (price = $0 so very easy to filter). Thank you for your input! And I would really appreciate the link 👍

honnesttrader commented 4 years ago

@lazyprogrammerio can you please share link to a public web app @nmapx Great work thanks for the list great help for me.

nmapx commented 4 years ago

The symbols are now compared with external API. The risk of ticker mismatch was reduced significantly. 😄 Closing the issue.