LDAR-Sim / LDAR_Sim

MIT License

11 stars 20 forks source link

V4 assorted bugs #191

Closed SEAJang closed 4 months ago

SEAJang commented 4 months ago

Pull Request Key Information

Reason for change

Fixed assorted bugs encountered while running a more complex simulation

What was changed

Improved code performance in program_outputs
Type casted values for datetime.timedelta
Added name validation

Intended Purpose

Fix assorted bugs and improve code performance

Testing Completed

Manually ran simulation with valid and invalid names.

All unit tests pass result.txt

SEAJang commented 4 months ago

All the comments regarding typecast - the values can sometimes be int32/int64. Timedelta is only valid with int and will error out if it's using int32/64, and that is why the typecasts have been added in.