Skip to content

Update from Master#45

Merged
mrpozzi merged 11 commits into
devfrom
master
Oct 30, 2017
Merged

Update from Master#45
mrpozzi merged 11 commits into
devfrom
master

Conversation

@mkao006

@mkao006 mkao006 commented Oct 30, 2017

Copy link
Copy Markdown
Contributor

This was due to an incorrect PR from a branch directly to master.

mkao006 and others added 11 commits October 5, 2017 10:27
…thon

There were minor differences between the implementation, the difference was actually
due to the default setting of noramlisation of the response. It is defaulted to True in R,
while False for Python. Normalisation usually improves the fit and prediction, this is
also supported by the examination of the prediction. Thus, we have adjusted both to
standardise the response
Note, the RNN implementation is very crude, and requires further finetuning
1. We extended the input data by the timestep size, so the prediction starts on the same day as other models.
2. the forecast input is padded so that the forecast give the right length of output.
@mrpozzi mrpozzi merged commit 3c3487d into dev Oct 30, 2017
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

Successfully merging this pull request may close these issues.

2 participants