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

Any progress #4

Open
zhuygln opened this issue Apr 20, 2019 · 1 comment
Open

Any progress #4

zhuygln opened this issue Apr 20, 2019 · 1 comment

Comments

@zhuygln
Copy link

zhuygln commented Apr 20, 2019

Hi Marco,
Have you got any update about this replicating this?
I find this work very interesting for the probability to apply the method to nucleosynthesis network calculation.
Look forward to hearing more from you.
Yonglin Zhu
Ph.D. Candidate

@mlpanda
Copy link
Owner

mlpanda commented Sep 3, 2019

Hi Yonglin,

Not really as I am confident that the authors have made a mistake (probably used the WT on the entire time-series, hence peaking into the future). That said, I do think that parts of the paper makes sense for other applications, it's just not feasible to create those returns on daily S&P prices.

PS. have a look at this comment from a user on reddit: https://www.reddit.com/r/StockMarket/comments/cr9ncs/ive_reproduced_130_research_papers_about/

He mentions this paper:

"The most frustrating paper:

I have true hate for the authors of this paper: "A deep learning framework for financial time series using stacked autoencoders and long-short term memory". Probably the most complex AND vague in terms of methodology and after weeks trying to reproduce their results (and failing) I figured out that they were leaking future data into their training set (this also happens more than you'd think)."

All the best,
Marco

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