-
Notifications
You must be signed in to change notification settings - Fork 12
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
Error when passing non-default date ranges to calc_suite_specs
#295
Comments
@jdossgollin thanks for the issue and nice example -- it looks like our documentation is indeed wrong here. If you're up for it, a pull request to fix it would be welcome! Changing |
@spencerkclark sounds good. This will be my first PR on someone else's project so apologies in advance if I make a mess of it 😄 . |
No worries -- we're happy to help out if you hit any snags! |
@spencerkclark which branch should I try to merge into? |
Good question -- we use the |
Thanks @jdossgollin for catching this and the other things you've brought to our attention. As @spencerkclark said, don't worry at all about being new. We're 100% happy (thrilled even) to get any question, bug reports, and PRs, from anybody. |
When I pass a custom date range, I get an error
TypeError: 'datetime.datetime' object is not subscriptable
. It's coming fromcalc.py
:Here's an MWE:
and the full error
The text was updated successfully, but these errors were encountered: