Skip to content

Update InputEqn.m #6

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Update InputEqn.m #6

wants to merge 1 commit into from

Conversation

GinaSierra
Copy link
Contributor

@GinaSierra GinaSierra commented Dec 13, 2017

This function uses a segment-based matrix of size (n_segments)x(n_samples) which is constructed in the first call to InputEqn (or when inputParameters changes). Precomputing this matrix provides a decrease in the prediction processing times because then it is not recomputed at each call to InputEqn. However, because it is necessary to find the indexes for each sample, the processing time still is higher than the function using the time-based matrix where instead of looking for the indexes in the matrix, uses the input time as an index.

The figure shows the prediction processing times using the original InputEqn function, the time-based matrix function and the extension of the original function. Although the processing times with the modified segment-based matrix function are higher than the processing times with the time-based matrix function, the modified segment-based matrix function provides a reasonable tradeoff between processing time and memory resources.
predictionprocessingtime

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.

1 participant