Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REopt API call fails for PVWatts cases #1290

Closed
cpaulgilman opened this issue Dec 1, 2022 · 2 comments · Fixed by NREL/ssc#952 or #1301
Closed

REopt API call fails for PVWatts cases #1290

cpaulgilman opened this issue Dec 1, 2022 · 2 comments · Fixed by NREL/ssc#952 or #1301
Assignees
Labels

Comments

@cpaulgilman
Copy link
Collaborator

cpaulgilman commented Dec 1, 2022

See https://sam.nrel.gov/forum/forum-general/4085

To replicate:

  1. Create a default PVWatts-Battery / Commercial case.
  2. On the System Design page, click Get battery size and dispatch from REopt.
  3. See UI callback error.
Could not evaluate callback function:call_reopt->on_change
[30]: error in call to 'reopt_size_battery()': Invalid inputs. See 'input_errors'.
[ Invalid length for prod_factor_series_kw. Samples must be hourly (8,760 samples), 30 minute (17,520 samples), or 15 minute (35,040 samples) ][11]: eval error in statement list
@cpaulgilman
Copy link
Collaborator Author

@brtietz Did this PR break "simple" REopt call for PVWatts-Battery: NREL/ssc#925?

@brtietz
Copy link
Collaborator

brtietz commented Dec 2, 2022

Yes, that seems likely, though it's not clear why. I can add it to my list for the first patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants