NREL / SAM

System Advisor Model (SAM)
BSD 3-Clause "New" or "Revised" License
345 stars 142 forks source link

REopt API call crashes SAM for weather file with subhourly data #1671

Closed cpaulgilman closed 8 months ago

cpaulgilman commented 10 months ago

Clicking Get size and dispatch on Battery Cell and System page should check weather file time step and display UI message instead of proceeding with API call.

Tested with 15-minute data from NSRDB.

brtietz commented 10 months ago

The REopt documentation suggests it should work with 30 and 15 minute data. While we should add a UI message for other subhourly timesteps, the fact that 15 minute data fails suggests a bug in how SSC sets up the API call.

cpaulgilman commented 10 months ago

The REopt documentation suggests it should work with 30 and 15 minute data. While we should add a UI message for other subhourly timesteps, the fact that 15 minute data fails suggests a bug in how SSC sets up the API call.

Agreed. Here are files for testing: reopt-subhourly-tests.zip