Skip to content
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

We need to specify date format for arrow.get() #5

Open
and-rewsmith opened this issue Mar 9, 2018 · 1 comment
Open

We need to specify date format for arrow.get() #5

and-rewsmith opened this issue Mar 9, 2018 · 1 comment

Comments

@and-rewsmith
Copy link
Collaborator

https://github.com/bshaw19/Crypto_Trader/blob/2f0f416c78b917d00f99b7c2ec0babcfbf33bc7e/Variable_Functions/Twitter_Sentiment.py#L16

@bshaw19 I'm not sure what date format you intend to use here. When I wrote this code for the LSTM stock model the input date worked without specifying a formatting argument to the arrow.get() function.

I was running this today and ran into problems with the "YYYY-MM-DD" format as without a formatting argument, it assumes another format and thus returns tweets for the wrong dates. When we know our date format we can add the argument to avoid this problem.

Arrow documentation: http://arrow.readthedocs.io/en/latest/

@bshaw2019
Copy link
Owner

We can specify it to match up with the Data_Grabber.py database date format for now. Trying to build "Inputs_Table_Builder.py" so that it automatically backfills based on the timestamps in the database.

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

No branches or pull requests

2 participants