search

KevWK314 / ThreeFourteen.AlphaVantage

.NET library for Alpha Vantage
MIT License
9 stars 9 forks source link

Added ability to capture split coefficient from json #1

Closed irperez closed 5 years ago

irperez commented 5 years ago

When pulling stock data for daily time series adjusted, the split coefficient was not being captured by any models. I've added the field. it is ignored when its a monthly or weekly time series adjusted and defaults to 0. I've tested using existing tests and added new assertions in those scenarios just mentioned.

Without the split coefficient, we cannot tell when and how the stock was split.

By the way, this is a nice library you have created! I like your design 👍

KevWK314 commented 5 years ago

Looks good Ivan. I'll create a new version.