forked from NREL/GEOPHIRES-X
-
Notifications
You must be signed in to change notification settings - Fork 1
Royalties [v3.9.59] #106
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
Closed
Closed
Royalties [v3.9.59] #106
Changes from 36 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
4146e30
Initial implementation of royalty rate. WIP - TODO to implement royal…
softwareengineerprogrammer 0b45fbe
Add entry in SAM Economic Models docs
softwareengineerprogrammer a20d3e4
Regenerate schema (with Royalty Rate)
softwareengineerprogrammer e175e95
Mark TODO remove or clarify project payback period: https://github.co…
softwareengineerprogrammer 94b8873
Include royalties in opex output/total opex (WIP)
softwareengineerprogrammer 59476a0
Regenerate schema with Royalties output param
softwareengineerprogrammer e0617fb
Assert that opex line items, including royalties, sum up to total
softwareengineerprogrammer 0aed3c2
Bump version: 3.9.54 → 3.9.55
softwareengineerprogrammer 883ec0c
update unit test
softwareengineerprogrammer a8defad
Change output param name to 'Average Annual Royalty Cost'
softwareengineerprogrammer 8459931
Fix incorrect method of setting royalties output param value
softwareengineerprogrammer 39dbf39
Internally distinguish between SAM econ royalty cost time series and …
softwareengineerprogrammer a102aa4
WIP - calculate Royalty Holder NPV
softwareengineerprogrammer af65dcb
assert royalty holder NPV
softwareengineerprogrammer 1c1deeb
finish impl of royalty holder outputs
softwareengineerprogrammer 3b8ce49
Update documentation. Throw exception if royalty rate provided for no…
softwareengineerprogrammer f5e5b35
SAM-EM royalties documentation (parameter mapping + dedicated section)
softwareengineerprogrammer 969f547
Bump version: 3.9.55 → 3.9.56
softwareengineerprogrammer c583cec
Move royalty outputs to EXTENDED ECONOMICS
softwareengineerprogrammer 5dd16d2
Outputs py38 future annotations
softwareengineerprogrammer c5fe65b
Royalty escalation rate + max (WIP to add unit tests, implement full …
softwareengineerprogrammer 71116b4
Mark TODO support custom royalty rate schedule as a list parameter
softwareengineerprogrammer fdddcd9
test_royalty_rate_schedule
softwareengineerprogrammer 4cac77b
move logic to Economics.get_royalty_rate_schedule
softwareengineerprogrammer 85a323e
test_royalty_rate_escalation
softwareengineerprogrammer 63ea5b9
Regenerate schema
softwareengineerprogrammer 5dfcbc1
Update SAM-EM Royalties docs
softwareengineerprogrammer 96d0ce4
example_SAM-single-owner-PPA-4: 50 MWe with Royalties
softwareengineerprogrammer 80c255c
Bump version: 3.9.56 → 3.9.57
softwareengineerprogrammer eaa9eb9
Add example_SAM-single-owner-PPA-4 to README examples table
softwareengineerprogrammer 2005c94
SAM-EM doc web interface links
softwareengineerprogrammer 50e6f97
Bump version: 3.9.57 → 3.9.58
softwareengineerprogrammer a782cd7
test_royalty_rate_with_addon
softwareengineerprogrammer a0d0c38
Clarify that royalty holder revenue/NPV are pre-tax
softwareengineerprogrammer b3ffad8
regnerate schema
softwareengineerprogrammer b23a6fe
Bump version: 3.9.58 → 3.9.59
softwareengineerprogrammer a60199c
slice pre-revenue years based on construction years (instead of hardc…
softwareengineerprogrammer 8b2d3af
Fix unit conversion issue that was causing incorrect royalty calculat…
softwareengineerprogrammer b1752e6
Clean up unit conversion
softwareengineerprogrammer File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
[bumpversion] | ||
current_version = 3.9.54 | ||
current_version = 3.9.59 | ||
commit = True | ||
tag = True | ||
|
||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.