-
Notifications
You must be signed in to change notification settings - Fork 23
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
15 fmp provider #16
base: master
Are you sure you want to change the base?
15 fmp provider #16
Conversation
1baf982
to
07e4aa7
Compare
d2f9d45
to
bb67891
Compare
bb67891
to
df7cf6d
Compare
GIve me a moment to update this PR with changes from master. In addition I want to make changes so that the training data and validation data can make use of as much data as available from the provider (see also #21 ) |
f1c5518
to
e04df2c
Compare
hey :) @jrdi @jaapterwoerds I thought about adding AlphaVantage support to opensignals. Thanks! |
hey, imo it would be better if we first do a PR refactoring of the code and move the required code to a common place, then we can start working on other changes adding more providers. |
Here a first version of an implementation of an data provider for https://financialmodelingprep.com/. I factored out most code so that implementing a data provider is basically implementing one function that interacts with the data provider's API.
Feedback is appreciated!
Closes #15