RMI-PACTA / workflow.data.preparation

The goal of `workflow.data.preparation` is to prepare all of the necessary data inputs for the Transition Monitor web application.
Other
2 stars 0 forks source link

reconsider the 2023Q4 parameters once they have been thoroughly investigated #176

Closed cjyetman closed 5 months ago

cjyetman commented 8 months ago

136 added the 2023Q4 config under time pressure with inadequate validation of the parameters. Once they have been thoroughly validated, we should update the config with the most correct values we can figure out.

AB#10387

jdhoffa commented 8 months ago

For posterity regarding scenarios:

Things that were flagged on Teams and ADO but may not have been formally addresse:

cjyetman commented 8 months ago

actually, decision for scenarios is supposed to be here, no? https://dev.azure.com/RMI-PACTA/2DegreesInvesting/_workitems/edit/9355/

jdhoffa commented 8 months ago

I'm not sure where it is "supposed to be", I'm just saying where it is haha.

I had a call to solicit scenario decisions and documented all decisions as a comment, tagging all involved in the call. The result of that can be found here: https://dev.azure.com/RMI-PACTA/2DegreesInvesting/_workitems/edit/9975#30492700

Maybe a good idea to also add it to the ticket you linked?

jdhoffa commented 8 months ago

I've added a comment to a similar effect to the ADO ticket you linked, with the latest verbal decisions that I have a record of from the last "Scenario Decision Making" call

cjyetman commented 8 months ago

Not to be nit-picky, but I'd really like us to have a well-defined place where it is "supposed" to be recorded so that it's easy to go back to and verify if needed. Comments in various threads have, in the past, conflicted with each other, so I think it's advantageous to have one fixed, agreed place where the decision is made clear. (thanks for your effort towards that @jdhoffa)

cjyetman commented 5 months ago

@jdhoffa says (in the ADO ticket) that the 2023Q4 parameters have been thoroughly reviewed and verified.... closing