Prometeo-Pyrrha
Prometeo-Pyrrha copied to clipboard
Call-for-Code
Add gauge tests and tests for a second gas. Also finish off documentation.
Add gauge tests and tests for a second gas.
- Previously, the tests only tested the TWA exposure calculations, not the comparison of those exposures to the corresponding limits - i.e. the limit 'gauge' percentages. This change adds those tests.
- Previously, the tests only tested one gas - carbon monoxide. This change adds tests for nitrogen dioxide as well. We don't have real nitrogen dioxide data yet, so it's mocked up data.
Additional changes to support these tests: 3. Added gas-specific rounding factors as carbon monoxide and nitrogen dioxide have different ranges. 4. Added the ability to let the tests override the windows & limits configurations, as the test results are all tied to a specific configuration. 5. Added CAL-OSHA, NIOSH and AEGL-2 example configurations to show how configuration might differ for other geographical regions, such as the NIOSH/ACGIH, CAL-OSHA or AEGL-2. This also facilitated confirming that the tests should fail (and do) when the wrong configuration is used. 6. Finished off outstanding documentation and dealt with most of the TODO's. 7. Modified DB field names to make them less sensitive to configuration changes. Fields for the different time-window TWAs are now named the after the number of minutes in that window (stable) and not by the display name (less stable and prone to SQL issues).
PR https://github.com/Code-and-Response/Prometeo-Rules-Decision/pull/9
@lidderupk - I added the DB field name change to this issue & associated PR.
- Modified DB field names to make them less sensitive to configuration changes. Fields for the different time-window TWAs are now named the after the number of minutes in that window (stable) and not by the display name (less stable and prone to SQL issues).