You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When loading tables from Coda, Stampy assumes they're going to contain columns with particular names.
There are 2 main ways this is problematic
Hiding a column in Coda UI also hides it from API requests, so when Stampy tries to access that column, it causes a KeyError. This has already happened. Solution: Make Stampy download the entire table or, even better, some subset of its columns, regardless of what is visible in the UI
Changing the name of a column causes the same kind of problem. This hasn't happen yet but might in the future. Solution: Make Stampy use column IDs which are immutable, so changing column names will not cause any problems
column ids can be deleted too, but I guess there is lower probability of unintentional damage (assuming accidental renaming is easier than deletion in Coda) + we already use table/view ids instead of table/view names => switching from column names to ids sounds OK
the main change to avoid Coda bugs could be to start using the raw Coda table with id grid-sync-1059-File with all columns instead of the Coda UI view with id table-YvPEyAXl8a that only shows the UI-visible columns from https://coda.io/d/_dfau7sl2hmG#_tutable-YvPEyAXl8a and is being used currently for
but the top priority should be make stampy not crash on Coda errors IMHO (or do we always want to crash on initialization Coda errors, just make those errors visible in #269? I think it shouldn't crash completely, but not a strong opinion)
When loading tables from Coda, Stampy assumes they're going to contain columns with particular names.
There are 2 main ways this is problematic
KeyError
. This has already happened.Solution: Make Stampy download the entire table or, even better, some subset of its columns, regardless of what is visible in the UI
Solution: Make Stampy use column IDs which are immutable, so changing column names will not cause any problems
For reference, here is the library we use to interact with Coda: https://github.com/Blasterai/codaio
and here is Coda API documentation: https://coda.io/developers/apis/v1#section/Using-the-API
The text was updated successfully, but these errors were encountered: