LDAR-Sim / LDAR_Sim

MIT License
11 stars 22 forks source link

<UPD>: Update default parameter data types #240

Closed SEAJang closed 4 months ago

SEAJang commented 4 months ago

Pull Request Key Information

Reason for change

Parameters that should have a float datatype were set to int datatype. This change updates the default parameter files to have the correct datatype.

What was changed

Update the existing parameters to have the correct data type

Intended Purpose

Prevent the bug where values when set to a float prevented the simulation from running.

Level of version change required

Patch

Testing Completed

Manually tested that simulation ran when costs and duration factor were set to both an integer and a float.

All unit tests pass. results.txt