LDAR-Sim / LDAR_Sim

MIT License
11 stars 20 forks source link

Optimize activate emissions #281

Closed ThomasGalesloot closed 1 month ago

ThomasGalesloot commented 1 month ago

Pull Request Key Information

Reason for change

Profiling demonstrates that the activate_emissions methods is a bottleneck in simulation runtime.

What was changed

This change optimizes the activate_emissions method at the component and source level.

Intended Purpose

Improve simulation runtime

Level of version change required

Patch

Testing Completed

All E2E tests pass: V4_simple_non_repairable_emissions_c8ea087f4a869534526cbd2cad4732963bda51a5.log V4-simple_repairable_emissions_c8ea087f4a869534526cbd2cad4732963bda51a5.log

All unit tests pass: unit_test_results.txt

Profiling shows moderate improvements: benchmark1_results_full_opt2.txt benchmark1_results_master.txt benchmark1_results_master_2.txt benchmark1_results_full_opt.txt

Target Issue

N/A

Additional Information

N/A