LDAR-Sim / LDAR_Sim

MIT License
11 stars 22 forks source link

<ENH>: Remove mitigation plots when no mitigation #223

Closed SEAJang closed 4 months ago

SEAJang commented 4 months ago

Pull Request Key Information

Reason for change

Plots that require mitigation values break when there is no mitigation (all emissions are non-repairable in simulation)

What was changed

Added checks for the cost to mitigation plot and the mitigation bar chart to ensure that they are only made when there are values to graph.

Intended Purpose

Remove visualizations that require mitigation numbers when there is no mitigation.

Level of version change required

Patch bump.

Testing Completed

All unit tests pass. results.txt

Ran manual simulation where there were only non-repairable emissions. Ran manual simulation where there were both repairable and non-repairable emissions.

Target Issue

Closes #222