2DegreesInvesting / tiltIndicator

Indicators for the TILT project
https://2degreesinvesting.github.io/tiltIndicator/
GNU General Public License v3.0
1 stars 1 forks source link

Review Physical Risk indicator code and assess level of maturity #767

Closed Tilmon closed 2 months ago

Tilmon commented 2 months ago

Hi @kalashsinghal,

can you please review the physical risk indicator code and assess the level of maturity. You can find detailed documentation here.

In essence, the indicator calculates physical risk exposure on company-level (no product-level results), based on a company's location. For that, the indicator

For more information, please refer to the documentation.

Thanks!

Pls check out Linda's files here:

kalashsinghal commented 2 months ago

@Tilmon I assessed the Physical risk code and came to the conclusion that the code still needs a lot of improvements. Currently, the code is non-functional. These are the improvements we need to work on:

  1. Resolve hidden bugs.
  2. Write unittests so that resolved bugs won't appear again.
  3. Convert current scripts and data into an R package.
  4. Document each function and process.

According to my judgement, Physical risk code needs 8-9 days time to be considered prototype-ready.

Tilmon commented 2 months ago

@kalashsinghal thanks for the info, good to know! We can discuss in the next sprint on how to proceed.