Skip to content

Fix connection costs not properly factored in or discounted#232

Merged
BenWinchester merged 4 commits intodevfrom
225-connection-costs-not-properly-factored-in-or-discounted
Apr 24, 2024
Merged

Fix connection costs not properly factored in or discounted#232
BenWinchester merged 4 commits intodevfrom
225-connection-costs-not-properly-factored-in-or-discounted

Conversation

@BenWinchester
Copy link
Contributor

Description

This pull request is being opened to address the issue whereby connection costs for households were only factored in for additional households added to the community in subsequent years and not for initial connections.

For example, with the default location of Bahraich, the community starts with 100 households and then expands throughout the simulation and optimisation periods. However, the cost for connecting these households to the mini-grid network is only included for the additional houses which are created throughout the lifetime of the simulation and not for the initial 100 households. Hence, all results in CLOVER under-report on the total cumulative system cost and the LCUE values.

This will be merged into the development branch and will be publicly released as part of CLOVER 5.2 or CLOVER 6. Hence, the version it is being merged into is 5.2.0a7.

Linked Issues

This pull request:

Unit tests

This pull request:

  • modifies the integration testing as all of the cumulative costs and LCUE values have increased as a result of including these additional hosueholds.

@BenWinchester BenWinchester added the bug Something isn't working label Apr 24, 2024
@BenWinchester BenWinchester added this to the CLOVER 6.0 milestone Apr 24, 2024
@BenWinchester BenWinchester self-assigned this Apr 24, 2024
@BenWinchester BenWinchester enabled auto-merge April 24, 2024 14:02
@BenWinchester BenWinchester merged commit 5d75d84 into dev Apr 24, 2024
@BenWinchester BenWinchester deleted the 225-connection-costs-not-properly-factored-in-or-discounted branch April 24, 2024 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants