LDAR-Sim / LDAR_Sim

MIT License
11 stars 20 forks source link

<FIX>: Update context manager use #150

Closed SEAJang closed 7 months ago

SEAJang commented 7 months ago

Pull Request Key Information

Reason for change

In python, using a context manager for I/O (with statements) are much safer than inline I/O

What was changed

Functionally the code did not change

Intended Purpose

Update the existing code to be more consistent and use context managers where applicable

Testing Completed

Compared the output summary csv files using FC after running the same simulations (generator files used)

Comparing files daily_summary_stats.csv and DAILY_SUMMARY_STATS1.CSV FC: no differences encountered

Comparing files emissions_summary_stats.csv and EMISSIONS_SUMMARY_STATS1.CSV FC: no differences encountered