2DegreesInvesting / tilt.company.match

MVP for company name matching in the tilt project
https://2degreesinvesting.github.io/tilt.company.match/
Other
0 stars 0 forks source link

Rename to `companies_id` for consistency with the tilt database #32

Open maurolepore opened 1 year ago

maurolepore commented 1 year ago

It's easy to forget to rename companies_id to id. It's easiest to be consistent with tiltData.

This affects all mentions of the id column in docs and datasets such as demo_loanbook, demo_tilt and friends.

I understand we already asked some banks to create loanbooks with the id column so this should be backward compatible or fail very gracefully.

maurolepore commented 1 year ago

Relates to #96 in that it's something that it affects the tilt dataset -- which BTW we should create in a systematic way maybe with an utility in this repo.