Skip to content

Conversation

@jhoodsmith
Copy link

Closes #662 by fixing #662.

The problem with #662 was that the reversing of the data happened before the data.loc[start_str:end_str]. Unless start_str = end_str, this would always return empty.

I could have equally made the same fix by changing data.loc[start_str:end_str] to data.loc[end_str:start_str].

@gliptak
Copy link
Contributor

gliptak commented Sep 16, 2019

@jhoodsmith it seems that the Aphavantage time-series API was reset #677

@jhoodsmith
Copy link
Author

In that case we can solve by revoking #662 and ignoring this pull (#679) - i.e., by removing the line data = data[::-1].

@bashtage
Copy link
Contributor

Going to close. Can reopen if this is still an issue. Thanks.

@bashtage bashtage closed this Sep 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AlphaVantage raises "Please input a valid date range" for previously working code

3 participants