search

LDAR-Sim / LDAR_Sim

MIT License
11 stars 20 forks source link

V4 oputputs processing overhaul #174

Closed ThomasGalesloot closed 5 months ago

ThomasGalesloot commented 5 months ago

Pull Request Key Information

Reason for change

Previous outputs logic was mixed in with other code and made it difficult to add new outputs.

What was changed

The outputs module was refactored to made modified outputs easy.

Intended Purpose

Improve the design of LDAR-Sim outputs logic

Level of version change required

N/A

Testing Completed

All unit tests pass on latest commit unit_test_results.txt

Some manual testing performed to ensure code runs and all expected visualizations are created

Target Issue

N/A

Additional Information

ThomasGalesloot commented 5 months ago

Latest E2E test results all passing: unit_test_results.txt