Skip to content
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.

Sign up for GitHub

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

Jump to bottom

clean up ForecastResolver #46

Open
nickderobertis opened this issue Apr 27, 2020 · 0 comments
Open

clean up ForecastResolver #46

nickderobertis opened this issue Apr 27, 2020 · 0 comments

Comments

@nickderobertis
Copy link
Owner

nickderobertis commented Apr 27, 2020 •
edited
Loading

ForecastResolver and associated logic is messy after reworking it multiple times.
Need to remove unneeded code and restructure more logic into classes. PlugResult
could handle more operations with the plugs, and the math could be more separated
from the finance logic.

This issue has been automatically created by todo-actions based on a TODO comment found in finstmt/resolver/forecast.py:20. It will automatically be closed when the TODO comment is removed from the default branch (master).
nickderobertis pushed a commit that referenced this issue Apr 27, 2020
@todo-actions
Update TODO references: #45, #46
f1b9080
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nickderobertis