Updatable inference lens #290
Open
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
…le_inference_lens # Conflicts: # src/main/scala/mimir/models/TypeInferenceModel.scala Fixing conflicts
# Conflicts: # src/main/scala/mimir/lenses/TypeInferenceLens.scala # src/main/scala/mimir/models/TypeInferenceModel.scala Merge conflicts
…mimir into updatable_inference_lens # Conflicts: # src/main/scala/mimir/models/TypeInferenceModel.scala
* master: fix other issues created by fixing jai_core unresolved dependency bug in build Finally fixed jai_core unresolved dependency bug in build Finally fixed jai_core unresolved dependency bug in build Finally fixed jai_core unresolved dependency bug in build Add udf and exec mode to dump the domain for vgterms as json. Update README.md Update README.md add missing files hacking in domain dumper Use hard table for missing keys instead of creating a table in the backend. Bug fix in feedback spec Additional bug fix in detect series for inputs with less than three rows. Bug fix in DetectSeries - it now does not explode if there are less than three rows returned by a query.
|
I will also note that likely the sanity checker should be scheduled for nightly testing, if possible. |
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.
Implemented the updateable inference lens and the progressive update trait, testing for short and longer updating, added model manager to update models in a background thread. The model manager will have a background thread that repeatedly checks each model with the update trait if that model is updated. If it hasn't completed updating, this thread calls the model to update itself.