idaholab / HERON

Holistic Energy Resource Optimization Network (HERON) is a modeling toolset and plugin for RAVEN to accelerate stochastic technoeconomic assessment of the economic viability of various grid-energy system configurations, especially with application to electrical grids and integrated energy systems (IES).
Apache License 2.0
22 stars 36 forks source link

[DEFECT] Taxes & Inflation flags on recurring cashflows don't do anything #265

Closed dylanjm closed 1 year ago

dylanjm commented 1 year ago

Defect Description

Describe the defect

Currently, a user cannot prevent taxes from being applied to a recurring cash flow. According to TEAL/src/Cashflows.py#L1127, any recurring cash flow is automatically taxed by default.

If a user specifies taxable=False on a <Cashflow> that is recurring, it will apply the tax rate anyway.

In HERON, users have the ability to specify if a cash flow is taxed or not, but in the code, we do not actually pass this information on to TEAL. See HERON/src/DispatchManager.py#L335-L340

This becomes an issue, for example, if a user wants to implement a Production Tax Credit (PTC) for a specific resource on a specific component. This is a recurring cash flow that should not be taxed.

What did you expect to see happen?
What did you see instead?
Do you have a suggested fix for the development team?

Describe how to Reproduce Steps to reproduce the behavior:

Screenshots and Input Files Please attach the input file(s) that generate this error. The simpler the input, the faster we can find the issue.

Platform (please complete the following information):


For Change Control Board: Issue Review

This review should occur before any development is performed as a response to this issue.


For Change Control Board: Issue Closure

This review should occur when the issue is imminently going to be closed.

dylanjm commented 1 year ago

Approved to close via #131