-
Notifications
You must be signed in to change notification settings - Fork 3
New price strategies + allow to vary by commodity #1021
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
Merged
Merged
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
8e9060d
Basic approach to allow commodity-dependent pricing strategy
tsmbland ffefd7d
Add MarginalCost pricing strategy
tsmbland 5bb3260
Add FullCost pricing strategy
tsmbland 12f3876
Small tidy-ups
tsmbland 6c15425
Less wasted price calculation
tsmbland 21af6a6
Fix tests
tsmbland 91f617e
Add unpriced method
tsmbland 7d20913
Add "default" option
tsmbland bcda35d
Restore warnings
tsmbland d7a652b
Fix calculation errors
tsmbland 3c68f83
Get working for assets with multiple outputs
tsmbland 0c418dd
Small tidy ups
tsmbland 4f1d351
Fix mistake with multiple output commodities
tsmbland efdbb0b
Update schema
tsmbland 93cd72e
Add tests for validate_commodity
tsmbland c8d3008
Caching capital cost for more efficient price calculation
tsmbland 0429d75
Merge branch 'main' into pricing_strategy
tsmbland d708aee
Use candidates for new strategies
tsmbland 6f8f6f1
Fix for some missing prices
tsmbland f7cd18f
Performance improvements
tsmbland fba830a
Merge branch 'main' into pricing_strategy
tsmbland e4fe4fa
Apply output levies and flow costs to the specific commodity
tsmbland e61bf68
Move some code to assets submodule
tsmbland c937eb8
Use candidates only as backup
tsmbland 7de4257
Only consider existing assets with activity
tsmbland aff8bb2
Revert some changes
tsmbland 89c8f82
Docstrings
tsmbland 46c362c
Update example models to use full cost method
tsmbland 9c6c408
Update circularity model
tsmbland 80659cd
Docstrings and small improvements
tsmbland 5a438f2
Add examples to docstrings
tsmbland 80f4b1f
Fix to take availability limits into account
tsmbland f12ce18
Fix tests
tsmbland 19cb06b
Update regression test data
tsmbland 456ad27
Prevent unnecessary calculation of activity limits
tsmbland 11f60de
Suggestions from code review
tsmbland 6e3fa57
Update tests
tsmbland 1a74dfc
Remove Default option from PricingStrategy
tsmbland 3f2fc6b
Simplify code for assembling pricing sets
tsmbland dcbc443
iter_marginal_costs_with_filter
tsmbland 5bf3988
EPSILON constant for unit types
tsmbland 26908e0
Move broken options check to input layer
tsmbland f6d0d6a
Add item type
tsmbland 6c6d6d6
Add tests for new pricing methods
tsmbland 678c498
Revert "Update example models to use full cost method"
tsmbland 1eb56b1
Leave pricing strategy unspecified
tsmbland e6a88d2
Update output files
tsmbland af30546
Merge branch 'main' into pricing_strategy
tsmbland 22a7b1a
Merge branch 'activity_limits_bug_fix' into pricing_strategy
tsmbland 68e2f3d
Remove empty pricing_strategy columns from commodities files
tsmbland 7c6370b
Update documentation
tsmbland dc7a617
Gatekeep new pricing strategies
tsmbland 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
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.
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.