NREL / edv_synthetic_smart_meter_data

Other
1 stars 1 forks source link

occupant variability leap year issue #142

Open JanghyunJK opened 3 years ago

JanghyunJK commented 3 years ago

@tsbyq quick question. was there any improvements made on the occ var gem for this (below) leap year issue we spotted before?

https://github.com/NREL/edv_synthetic_smart_meter_data/blob/61103f2b7f57da779ffd6a1a57f9de860725b6bc/scripts/helper/simulate_bdgp_xml_path.rb#L26

tsbyq commented 3 years ago

@JanghyunJK I checked the latest occupancy simulator - it could run simulation for the correct number of days for leap years. However, the reported date is wrong (it shows 12/32 instead of 2/29). I've reached out to our occupancy simulator developer to see if we can fix this quickly.