LDAR-Sim / LDAR_Sim

MIT License
11 stars 22 forks source link

feat: Refactor LDAR-Sim main script #282

Closed ThomasGalesloot closed 3 months ago

ThomasGalesloot commented 3 months ago

Pull Request Key Information

Reason for change

The LDAR-SIm main script became too long and hard to maintain.

What was changed

This change refactors the main script into smaller method using the new SimulationManager class to maintain state through class variables.

Intended Purpose

The main script will now be easier to read and maintain as it is no longer monolithic.

Level of version change required

N/A

Testing Completed

Manually ran simple test case 1 on master and after the change and used E2E testing code to compare results, confirmed results were identical: main_vs_new_testing.zip

All unit tests pass: unit_test_results.txt

All End to End tests pass: V4_simple_non_repairable_emissions_dac922076552adcca422bf0c3471343e22073a70.log V4-simple_repairable_emissions_dac922076552adcca422bf0c3471343e22073a70.log

Target Issue

N/A

Additional Information

N/A