microsoft / times-excel-reader

MIT License
6 stars 3 forks source link

Tags missing #17

Open olejandro opened 1 year ago

olejandro commented 1 year ago

@Antti-L noticed that the tag list is missing the following tables: TFM_MIG and TFM_DINS_TS. Is it the list in times_mapping?

olejandro commented 1 year ago

Ah, I see, they should be added to TODO.md

Antti-L commented 1 year ago

That should read TFM_DINS-TS, just like TFM_INS-TS (dash vs. underscore). And note that TFM_MIG (like TFM_UPD) supports the column Sourcescen, which tells where to look for the source data to be updated or migrated. This is important, because the model may have many scenario files defining data for the same parameters, and for controlling the end-result of the updating or migrating scenario, one therefore needs a way of pointing to the source scenario as well. Moreover, the data generated by TFM_UPD or TFM_MIG can subsequently be the source for another TFM_UPD or TFM_MIG operation, in a later scenario, while that later scenario may equally well also refer to the original data (e.g. Base). The order of the scenarios is thus crucial also in this respect (order is defined in the RUN file, and also in the case json files under VEDA), like it is crucial for the correctness of basically all of the model data.

olejandro commented 1 year ago

Thanks @Antti-L! We've started working on the file containing info about the tags in samwebster#39. Please see if you have any comments.