Added C++ REST API #497
Merged
Added C++ REST API #497
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 30 commits
May 10, 2021 15:04
The external function should provide the derivative at each time point.
This is the dumbest model. Each root node predict close to the average of the observation series.
or any time series data.
…ation fault happening on old test code
1. All the programs compile. 2. All the tests pass. However all the executions are not correct. The head node modeling components are not properly integrated with CauseMos.
1. Need to do more: a. Cleaning b. Testing
1. Remember the sequence of log likelihoods throughout the sampling process including burning. 2. Complete state now includes the log likelihood sequence. 3. Plotting function now plots the log likelihoods.
1. Added code to identify and retain the MAP sample even if we encounter during the burning period. 2. If we encounter the MAP sample within the burning period it is always put at res - 1 position of the retained samples and we reduce the number of samples retained during the sample resolution period by 1. 3. Finally delphi remembers the index of the MAP sample withing the retained samples. Note: There are some (or more) repeated code in this push and previous ones that needs to be cleaned up when there is more time.
The seasonality was not visible when Delphi was invoked through the CauseMos interface. When a model is created, each concept’d period for seasonality is inferred from data. However, it was not saved into the serialized model. When creating an experiment, an AnalysisGraph object is created by deserializing the saved model. Since concept periods were not saved all the concepts ended up having the default period of 1.
Added some methods to generate a random CAG and populate it with synthetic observations.
1. Removed commented code 2. Moved a block of code to make some repeated code identical in two places. I did not refactor that into a separate method since it is only a few lines of code and those few lines alone does not create a coherent functionality.
This caused a chunk of repeated code. I feel that using a separate class to handle the delphi.db is a better approach.
Implemented and debugged. Based on the Fourier decomposition. As per my tests, it is working.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.