Pysam hybrids#1584
Conversation
sjanzou
left a comment
There was a problem hiding this comment.
Cannot build on Windows with following export_config externals error
restore develop as target for develop
sjanzou
left a comment
There was a problem hiding this comment.
All the Projects are build on Windows - awesome!
However, the Generic PVWatts Wind FuelCell Battery Hybrid configurations are failing on Windows, looks like "generic_degradation" needs to be updated e.g. (Host Developer)
As are the PVWatts Wind FuelCell Battery configurations with missing "degradation" for FuelCell configurations, e.g. (Single Owner)
Also, the Hybrid configuration test results are changing from the patch branch
All four hybrid with Fuel Cell configurations are still failing to run.
…or hybridized inputs only
Note that hybrid configurations will have to be reevaluated for changes before merging into patch
|
@sjanzou I merged in SAM_1515 but I'm still getting outputs like |
|
Same inputs giving different financial outputs, pysam_hybrids on left and patch on right |
sjanzou
left a comment
There was a problem hiding this comment.
We should resolve the differences in financial outputs for both ssc and SAM with the pysam_hybrids branches.
| @@ -60,8 +60,8 @@ Standalone Battery,Merchant Plant,-8397703,26.6181,NA,; Warning: IRR at end of a | |||
| Standalone Battery,Leveraged Partnership Flip,-8397703,78.2835,72.514,; Warning: IRR at end of analysis period is not a number (NaN). This can indicate that revenues are too low to cover costs, or that they are excessively high compared to costs. Warning: IRR in target year is not a number (NaN). This can indicate that revenues are too low to cover costs, or that they are excessively high compared to costs. Warning: NPV is $-3.12112e+07. A negative NPV indicates project costs are higher than revenues. | |||
| Standalone Battery,All Equity Partnership Flip,-8397703,79.1754,72.514,; Warning: IRR at end of analysis period is not a number (NaN). This can indicate that revenues are too low to cover costs, or that they are excessively high compared to costs. Warning: IRR in target year is not a number (NaN). This can indicate that revenues are too low to cover costs, or that they are excessively high compared to costs. Warning: NPV is $-3.85976e+07. A negative NPV indicates project costs are higher than revenues. | |||
| Standalone Battery,Sale Leaseback,-8397703,80.5079,72.514,; Warning: IRR at end of analysis period is not a number (NaN). This can indicate that revenues are too low to cover costs, or that they are excessively high compared to costs. Warning: NPV is $-4.61131e+07. A negative NPV indicates project costs are higher than revenues. Warning: NPV is $-8.70063e+07. A negative NPV indicates project costs are higher than revenues. | |||
| ETES,Single Owner,-466572064,14.7945,7.40855,; | |||
| PTES,Single Owner,-197595520,18.6636,10.3676,; | |||
| ETES,Single Owner,-467817888,14.8003,7.39191,; | |||
There was a problem hiding this comment.
Why are these differing? They are in the META tag for some o and m costs but are not hybridized...
There was a problem hiding this comment.
It may be that these just are non-deterministic. I just ran the PTES Single Owner case off patch and got different values from both of these:
-198005504, 18.65, 10.35
| PVWatts Wind FuelCell Battery Hybrid,Single Owner,579660736,10.6946,5.41105,; Notice: Simulation time step is 60 minutes for pvwattsv8. Notice: Simulation time step is 60 minutes for windpower. | ||
| PVWatts Wind FuelCell Battery Hybrid,Host Developer,2.68732e+06,49.7344,51.3994,; Notice: Simulation time step is 60 minutes for pvwattsv8. Notice: Simulation time step is 60 minutes for windpower. | ||
| Photovoltaic Wind Battery Hybrid,Single Owner,594070784,8.46988,5.95228,; Notice: Simulation time step is 60 minutes for pvsamv1. Notice: Simulation time step is 60 minutes for windpower. | ||
| PVWatts Wind Battery Hybrid,Single Owner,578883008,10.553,5.91853,; Notice: Simulation time step is 60 minutes for pvwattsv8. Notice: Simulation time step is 60 minutes for windpower. |
There was a problem hiding this comment.
All the hybrid financial outputs differ and so do the ssc test using the same input file as commented on ssc pull request 1091.
sjanzou
left a comment
There was a problem hiding this comment.
Working on windows and results matching with patch branches ssc and SAM.
Great work
Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
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.