Open mRaffill opened 1 year ago
(I already made this change, just making the issue to keep track of the status)
This change was completed in 9a229c88ef9bfc5a3a0922810518580a3a2b9f9b
The relative crash change does make most of the results negative, except for a few projects (which I should look into later)!
Next steps
Just realized I actually subtracted in the wrong order! It should be after crashes - before crashes, not before crashes - after crashes!! So actually a majority of the projects have a positive change (increase) in crashes, even with relative change.
This may just be because of the inconsistency between the model-based NC and user-inputted EC (since I believe a majority of projects do have user-inputted crashes)
Retry this with the results of the new crash equations:
So new To-Do list:
Just realized I actually subtracted in the wrong order! It should be after crashes - before crashes, not before crashes - after crashes!! So actually a majority of the projects have a positive change (increase) in crashes, even with relative change.
With the correct order, the graph of the current relative crash change actually looks like this (just a reflection of the previous graph - everything is actually positive!):
After using the crash equations where new crashes are calculated directly from existing crashes, in 138e0f5ad5b197631800785a59f9b687c471dec5, the relative crash change is (almost) all negative as expected!
model only relative crashes | user-input only relative crashes |
---|---|
(I am pretty sure the few positive instances are projects with a CRF >= 1, based on these graphs showing that before and after crashes are basically equal in these cases, but I should definitely look into this in more detail)
For projects with crash change > 0:
Currently, relative "before crashes per 1000 volume" and "after crashes per 1000 volume" are calculated, but the crash change still uses absolute (not per volume) crashes. We are considering displaying relative crash change in the tool instead to capture the change in "risk" (I think?). So to test this: