LDAR-Sim / LDAR_Sim

MIT License
11 stars 22 forks source link

fix: Fix failure to keep program outputs #291

Closed ThomasGalesloot closed 2 months ago

ThomasGalesloot commented 2 months ago

Pull Request Key Information

Reason for change

When running simulations, no programs outputs were being kept, which made it difficult to debug and understand the simulation results.

What was changed

The simulation manager now correctly keeps the outputs of the programs.

Intended Purpose

First 5 Program outputs are now properly kept

Level of version change required

Patch

Testing Completed

Manual testing.

All E2E tests pass: V4_simple_non_repairable_emissions_9cd3397da018723e0a7ec09ad2486988f39d8497.log V4-simple_repairable_emissions_9cd3397da018723e0a7ec09ad2486988f39d8497.log

All unit tests pass: unit_test_results.txt

Target Issue

N/A

Additional Information

N/A