> Still assuming forecast_period is in hours

This is no longer true - this needs to be updated to reflect the actual implementation (which does handle the unit conversion).