LDAR-Sim / LDAR_Sim

MIT License
11 stars 20 forks source link

Ldar 55 update sensitivity analysis run script #283

Closed ThomasGalesloot closed 1 month ago

ThomasGalesloot commented 1 month ago

Pull Request Key Information

Reason for change

The sensitivity analysis script was long, hard to maintain and did not share code with other scripts that "run" LDAR-Sim.

What was changed

Refactored the sensitivity analysis script to use a sensitivity analysis simulation manager that inherits core functionality from the base simulation manager.

Intended Purpose

This allows the sensitivity analysis simulation manager to inherit changes made to LDAR-Sim run functionality.

Level of version change required

N/A

Testing Completed

All unit tests pass: unit_test_results.txt

All E2E tests pass: V4_simple_non_repairable_emissions_16b14b330fafe1fd64849d1f0e5290e8b2f81192.log V4-simple_repairable_emissions_16b14b330fafe1fd64849d1f0e5290e8b2f81192.log

Manual testing comparing master and post change with sensitivity analysis example is identical: sensitivity_analysis_script_manual_testing.zip

Target Issue

N/A

Additional Information

N/A