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
Update Residential / Commercial Cash Flow to Excel for Battery and Fuel Cell #1545
Conversation
I tested this with PV-Battery Commercial and got the OLE automation error 3 times in a row, even after restarting SAM and renaming the file so it wasn't overwriting an existing Excel. Any recommendations for how to kick that error? Is a full rebuild necessary? |
@brtietz I sometimes get an OLE Automation error with Send-to-Excel with Equations, but it usually opens the appropriate Excel workbook and populates the inputs, so it's just an annoyance to have to close the error window. Is it preventing the workbook inputs from being populated for you? This has long been an issue for any SAM functions that interact with Excel. |
When the error shows up for me, it populates some but not all of the values, which causes things like LCOE and NPV to mismatch (which seemed critical to the review). |
@sjanzou Do you have any thoughts on how fixable this OLE Automation error might be? I had not noticed it was sometimes resulting in partial data being sent from SAM to Excel. Should we create a separate issue for this to investigate after the release? |
The OLE Automation error is from automtn.cpp in src/msw/ole folder in wxWidgets: According to Microsoft, the error can be caused by a reference issue I created issue #1563 |
I think this PR fixes #1403. For this test file, I get reasonably close NPV, real LCOE, and nominal LCOE: @brtietz Can we merge this into Develop and deal with the OLE Automation error separately with #1563? |
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 based on test file
Pull Request Template
Description
Modify residential_commercial.xlsx workbook for send-to-Excel with equations:
Send-to-Excel with equations values of LCOE and NPV for default PV-Battery / Commercial and Fuel Cell / Commercial configurations should now match values in SAM metrics table.
Fixes #1403
Type of change
Please delete options that are not relevant.
Checklist:
If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.