LDAR-Sim / LDAR_Sim

MIT License
11 stars 22 forks source link

feat: Add concat logic for sites csv batch concat #120

Closed ThomasGalesloot closed 1 year ago

ThomasGalesloot commented 1 year ago

Pull Request Key Information

Reason for change

Per site specific info is a useful output. For batch running, this can still be reporting as per site averages with some concatenation.

What was changed

Added per site concat logic for sites csvs.

Intended Purpose

Concat sites csvs per site

Level of version change required

N/A

Testing Completed

Manual testing - To reproduce, add the following code into sites_concat.py and use the attached zipped files inside the batch module in a folder named "test_data". Files were inspected for correctness.

concat_sites_files("LDAR_Sim/LDAR_Sim/src/batch/test_data/sites_output_0_P_OGI.csv", "LDAR_Sim/LDAR_Sim/src/batch/test_data/sites_output_1_P_OGI.csv", "LDAR_Sim/LDAR_Sim/src/batch/test_data/combined_test_data.csv")

concat_sites_files("LDAR_Sim/LDAR_Sim/src/batch/test_data/sites_output_0_P_OGI.csv", "LDAR_Sim/LDAR_Sim/src/batch/test_data/combined_test_data.csv", "LDAR_Sim/LDAR_Sim/src/batch/test_data/combined_test_data2.csv")

Test files: Testfiles.zip

Target Issue

N/A

Additional Information

N/A