-
Notifications
You must be signed in to change notification settings - Fork 151
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
Fixes #1574 #1575
Fixes #1574 #1575
Conversation
Looks like this is failing due to "No default value found for 'batt_dispatch_auto_can_fuelcellcharge' in external file (PVWatts Wind Battery Hybrid/Single Owner), using internal default" (this may affect more than just this hybrid) |
A second callback error in this configuration: Could not evaluate callback function:Battery Dispatch Automated FOM->on_load |
I was working on adding these checkboxes in another branch (#1566), I can merge that into this branch once done. |
@mjprilliman , the startup dependent vartable was set to fuel cell vartable (2) instead of the battery vartable (3) for the PVWatts-Wind-FuelCell-Battery / Single Owner configuration... please update in your branch as follows: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you!
Yes, just merged it
From: Matt Prilliman ***@***.***>
Sent: Friday, November 17, 2023 9:20 AM
To: NREL/SAM ***@***.***>
Cc: Mirletz, Brian ***@***.***>; Review requested ***@***.***>
Subject: Re: [NREL/SAM] Fixes #1574 (PR #1575)
CAUTION: This email originated from outside of NREL. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Can this be merged? It's need for configs to pass on develop with ***@***.***<NREL/ssc@047030c>.
-
Reply to this email directly, view it on GitHub<#1575 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABKGHYC5R66SP5KR3X3QKGLYE6FEDAVCNFSM6AAAAAA7MD3FJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJWG4YTINRZGY>.
You are receiving this because your review was requested.Message ID: ***@***.******@***.***>>
|
Goes with NREL/ssc#1089