-
Notifications
You must be signed in to change notification settings - Fork 391
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
WIP : Long short term memory prediction models for Time series Data #2721
base: develop
Are you sure you want to change the base?
Conversation
…into feature/LstmModel
Codecov ReportAttention: Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #2721 +/- ##
=============================================
- Coverage 56.01% 54.91% -1.09%
- Complexity 8197 8419 +222
=============================================
Files 2103 2182 +79
Lines 89017 91818 +2801
Branches 6560 6730 +170
=============================================
+ Hits 49854 50414 +560
- Misses 37445 39655 +2210
- Partials 1718 1749 +31 |
Very interesting and nice pull request! Do you have benchmarks values? |
…into feature/LstmModel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed
* @return a {@link Prediction} object | ||
*/ | ||
public static Prediction from(ValueRange valueRange, Interval interval, ZonedDateTime time, Integer... values) { | ||
return from(valueRange, buildMapDuoDecimus(time, values)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interval
is not checked here!
* @return a {@link Prediction} object | ||
*/ | ||
public static Prediction from(ValueRange valueRange, Interval interval, ZonedDateTime time, Integer... values) { | ||
return from(valueRange, buildMapDuoDecimus(time, values)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to discuss this "duodecimus" thing... it breaks a lot of assumptions (and Javadocs) if we suddenly have 5-minute predictions...
This Model uses the LSTM (Type of Recurring neural network ) algorithm for predicting the Consumption and Production active power times series data.