Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Update InputEqn.m#6

Open
GinaSierra wants to merge 1 commit intonasa:developnasa/PrognosticsModelLibrary:developfrom
GinaSierra:patch-1GinaSierra/PrognosticsModelLibrary:patch-1Copy head branch name to clipboard
Open

Update InputEqn.m#6
GinaSierra wants to merge 1 commit intonasa:developnasa/PrognosticsModelLibrary:developfrom
GinaSierra:patch-1GinaSierra/PrognosticsModelLibrary:patch-1Copy head branch name to clipboard

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.