Closed baoh0212 closed 2 years ago
I still have a change to make, don't merge it.
I still have a change to make, don't merge it.
Change has been made. It is ready to be merged.
Hi @baoh0212 it looks like you force-pushed some updates and that introduced conflicts. Let me know if I can be helpful in resolving that.
EDIT: even as I said that, you merged in devel. You can disregard this comment.
@baoh0212 I checked the branch, it looks like CashFlow_test_Outputs.xml
doesn't include generating the plots. Maybe that got unwritten during the force push?
Continued in #63.
Pull Request Description
What issue does this change request address?
Closes #38
What are the significant changes in functionality due to this change request?
Adding three plots to visualize the cash flows:
Donut chart
For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.