Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Get series with realtime#11

Open
elmotec wants to merge3 commits intomortada:master
base:master
Choose a base branch
Loading
fromelmotec:get_series_with_realtime

Conversation

elmotec
Copy link
Contributor

Hey@mortada, this is a PR to address the issue I mentioned in#9. That's only the part that handle realtime_start and realtime_end as first class citizens in get_series().

I should have get_dataframe() (to query multiple series in one call at specific times) ready sometimes in October.

@mortada
Copy link
Owner

please rebase

@elmotecelmotecforce-pushed theget_series_with_realtime branch fromef78d83 to45a1ac2CompareOctober 1, 2015 09:41
@elmotec
Copy link
ContributorAuthor

Rebased.

Removed usage of sys.stderr in mock exception which caused the stream tobe closed. This in turn failed the next test.  Simplified test_search toavoid the zip and the loop.
@elmotecelmotecforce-pushed theget_series_with_realtime branch from3fe6082 toaa1db9bCompareOctober 19, 2015 02:13
Arguments realtime_start and realtime_end in get_series() now cause apandas.DataFrame to be returned with pandas.MultiIndex for realtimedata.  Added simple test for the new feature and documentation.Added __init__.py in fredapi.tests so it's correctly interpreted asa package. Now we could revert to python setup.py test in .travis.yml.Fixed test_invalid_kwarg_in_get_series() as we sometimes get aTypeError and sometimes a ValueError. Seems that pandas passes throughwhatever exception it gets, might be a good reason for this so wefollow the same policy.Simplified comparison of dataframe output in tests.
@elmotecelmotecforce-pushed theget_series_with_realtime branch fromaa1db9b to095b9f6CompareOctober 25, 2015 01:18
9999-12-31 cannot be converted to pandas.Timestamp because it's too big.Reason it's prefereable to use pandas.Timestamp than datetime.datetimeis that the former can be used as an index whereas the second cannot.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@elmotec@mortada

[8]ページ先頭

©2009-2025 Movatter.jp