-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: Support for historical time series data #8
Conversation
@arvindrajan92 thanks for the PR (and sorry for the delay!) Would you mind adding to the README (and expanding the unit tests) to cover usage of (Also fixing up the failing flake8 lint errors!) Thanks! |
@mindmelting no worries, and you're welcome :) I have expanded the unit test with more test cases and updated README.md. The flake8 lint errors should be fixed now. Please let me know if you get any more errors. Thanks |
@mindmelting doesn't seem like an error I could fix from my end. hope you could take a look at it. thanks for spending some time looking into this - sincerely appreciate it :) |
Will take a look this weekend, thanks! |
@arvindrajan92 I have fixed the build issue - are you able to bump the version to |
@mindmelting done 😄 |
Hi @mindmelting, I am adding support for retrieving historical time series data without breaking
get_data()
method.