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.

Sign up for GitHub

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

Jump to bottom

some changes #21

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

some changes #21

wants to merge 4 commits into from

Conversation

megazone87
Copy link

  1. py2 to py3
  2. As google finance data is unstable, especially in china, "pandas_datareader" and its "quandl" api is used for data_fetcher
  3. data batching is changed:
  • previous: target of current sample is the last value of next sample in the same batch.
  • now: make taget seprate from input, by shuffle the samples in whole set, thus they don’t related within one batch.
  1. sort price by date from old to new date, thus the prediction is about recent date.
  2. pandas deprecated: sort => sort_value

Thank you for checking!

@megazone87
py 2to3
8ef6bc8 
besides the 2to3 changes, another important change is:
- previous: target of current sample is the last value of next sample in the same batch.
- now: make taget seprate from input,  by shuffle the samples in whole set, thus they don’t related within one batch.

and the results show this method still works.
@megazone87
deperated sort => sort_value
12a52f0
@megazone87
sort price by date from old to new
51d2788
@megazone87
Update data_model.py
2fc81f1 
30 => num_steps
Sign up for free to 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.
1 participant
@megazone87