search

USACE / chart-docs

2 stars 0 forks source link

[SCT-LL]: Review life loss logic in go-consequences #12

Open HenryGeorgist opened 8 months ago

HenryGeorgist commented 8 months ago

Background

life loss logic has been added to go-consequences. the methodology needs to be reviewed by someone capable of reading code and understanding (in great depth) the lifesim methodology

trietmnj commented 8 months ago

How about 2 people - 1 that can read code and 1 with the life loss background reviewing together?

HenryGeorgist commented 8 months ago

i think nick can do both, but a second set of eyes on code is always good.

HenryGeorgist commented 8 months ago

https://github.com/USACE/go-consequences/blob/main/lifeloss/lifeloss.go

trietmnj commented 8 months ago

@GitLutz

HenryGeorgist commented 8 months ago

we need to add @GitLutz to the appropriate team on USACE github so that he can be assigned.

trietmnj commented 8 months ago

Will need @GitLutz to update his profile to the org policy standard.

https://github.com/USACE/policies