Fix issues in historical analysis and custom result index #888
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
…mpose (opensearch-project#243) Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 2964522)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 74b644c)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit b25860d)
…#262) Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 3b1b845)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 96a0ab2)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit f95f52b)
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 13c0962)
…t#278) (opensearch-project#280) * Fix missing data call out if window delay is large The missing feature callout only shows when fetching feature data points based on the interval and time range, and if there are any empty points Previously, when I configured the window delay to 20 minutes, the missing data call out will show. The reason is that the window delay offset isn't being considered in the time range used in the feature fetching query, or in the displayed chart, such that it shows as empty, when it's actually expected to be empty based on the window delay.This PR fixed the bug by considering window delay in missing feature calculations. Testing done: 1. done e2e testing and verified the problem is fixed. 2. added unit tests. Signed-off-by: Kaituo Li <[email protected]> (cherry picked from commit 9d34ba2) Co-authored-by: Kaituo Li <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 0d74e1a)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 3b80bda)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 6f736d7)
Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit d973b8e)
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 45a56ee)
Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit 77c0931) Co-authored-by: Jackie Han <[email protected]>
…#306) * fix failed unit tests Signed-off-by: Jackie Han <[email protected]> * Fix parse result bug for feature attribution Signed-off-by: Jackie Han <[email protected]> Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit 7ace8b4)
…#310) (opensearch-project#311) Signed-off-by: Jackie Han <[email protected]> Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit 25431a0) Co-authored-by: Jackie Han <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 5dd0162)
Signed-off-by: Amit Galitzky <[email protected]> (cherry picked from commit 6417553)
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 7ec2eab)
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 6958004)
…rch-project#334) * Add windows platform in CI workflow Signed-off-by: Jackie Han <[email protected]> * Adjust text content for buidling on Windows Signed-off-by: Jackie Han <[email protected]> * Update long text message format in FormattedFormRow Signed-off-by: Jackie Han <[email protected]> * Update long text message format in FormattedFormRow Signed-off-by: Jackie Han <[email protected]> * Update long text message format in FormattedFormRow Signed-off-by: Jackie Han <[email protected]> * Update long text message format in FormattedFormRow Signed-off-by: Jackie Han <[email protected]> * Update long text message format in FormattedFormRow Signed-off-by: Jackie Han <[email protected]> * Update long text message format in FormattedFormRow Signed-off-by: Jackie Han <[email protected]> Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit d8af147) Co-authored-by: Jackie Han <[email protected]>
Signed-off-by: Jackie Han <[email protected]> Signed-off-by: Jackie Han <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit a2474d6)
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit 0ee7503)
Signed-off-by: Amit Galitzky <[email protected]>
opensearch-project#384) (opensearch-project#386) Signed-off-by: Jackie Han <[email protected]> Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit ecd0814) Co-authored-by: Jackie Han <[email protected]>
Signed-off-by: Tyler Ohlsen <[email protected]> (cherry picked from commit ae5efd8)
Signed-off-by: Eric <[email protected]> (cherry picked from commit 4c49c8c) Co-authored-by: Eric Wei <[email protected]>
…omponents (opensearch-project#826) (opensearch-project#842) (cherry picked from commit 3203e43) Co-authored-by: Miki <[email protected]>
* header redesign Signed-off-by: Jackie Han <[email protected]> * update snapshot Signed-off-by: Jackie Han <[email protected]> --------- Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit 399025a) Co-authored-by: Jackie Han <[email protected]>
Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]>
) (opensearch-project#847) * Changed context menus to be small Signed-off-by: Dan Dong <[email protected]> * Updated tabs to be small Signed-off-by: Dan Dong <[email protected]> * Updated paragraph text to be EuiText size="s" Signed-off-by: Dan Dong <[email protected]> * Updated semantic headers Signed-off-by: Dan Dong <[email protected]> * Updated primary buttons Signed-off-by: Dan Dong <[email protected]> * Reverted unintended changes Signed-off-by: Dan Dong <[email protected]> * Updated snapshot testing Signed-off-by: Dan Dong <[email protected]> * Fixed merge conflict Signed-off-by: Dan Dong <[email protected]> * Updated snapshot tests --------- Signed-off-by: Dan Dong <[email protected]> (cherry picked from commit 07908a0) Co-authored-by: Dan Dong <[email protected]>
…-project#852) Signed-off-by: Miki <[email protected]> (cherry picked from commit a426e41) Co-authored-by: Miki <[email protected]>
…pensearch-project#851) (opensearch-project#853) * Add Missing Value Imputation Options and Update Shingle Size Limit This PR introduces a new missing value imputation feature with three options: zero, fixed values, and previous values. When the fixed values option is selected, users can input custom values for each feature. Validation logic has been added to ensure that the feature names and the number of custom values match the number of enabled features. Additionally, the review page and model configuration page have been updated to properly display these new parameters. This PR also increases the maximum shingle size to 128, aligning with the backend implementation. Testing: * Updated existing unit tests to reflect these changes. * Conducted manual end-to-end testing. Signed-off-by: Kaituo Li <[email protected]> * remove console log Signed-off-by: Kaituo Li <[email protected]> --------- Signed-off-by: Kaituo Li <[email protected]> (cherry picked from commit 3ed0058) Co-authored-by: Kaituo Li <[email protected]>
…t#859) (opensearch-project#860) This PR introduces suppression anomaly rules under the Advanced Settings section, enabling users to suppress anomalies based on the difference between expected and actual values, either as an absolute value or a relative percentage. Testing: * Added unit tests to verify the suppression rules functionality. * Conducted manual end-to-end (e2e) tests to validate the implementation. Signed-off-by: Kaituo Li <[email protected]> (cherry picked from commit 9874c48) Co-authored-by: Kaituo Li <[email protected]>
…-project#830) (opensearch-project#833)" (opensearch-project#862) * Revert "Add Enable Flattened custom result index checkbox (opensearch-project#830) (opensearch-project#833)" This reverts commit f53450c. * update snapshot Signed-off-by: Jackie Han <[email protected]> --------- Signed-off-by: Jackie Han <[email protected]>
Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit b06f1f7) Co-authored-by: Jackie Han <[email protected]>
…ct#854) (opensearch-project#863) (opensearch-project#867) (cherry picked from commit 4942f5f) Co-authored-by: Amit Galitzky <[email protected]>
Signed-off-by: Jackie Han <[email protected]> (cherry picked from commit 12fd1b2) Co-authored-by: Jackie Han <[email protected]>
…ect#849) (opensearch-project#873) * Add generate anomaly detector action to discover page Signed-off-by: gaobinlong <[email protected]> * Add more test code and rename the file Signed-off-by: gaobinlong <[email protected]> * Modify flyout header Signed-off-by: gaobinlong <[email protected]> * Make the detectorName follow the convention Signed-off-by: gaobinlong <[email protected]> * Truncate the index pattern name if it's too long Signed-off-by: gaobinlong <[email protected]> * Move entry point to query editor Signed-off-by: gaobinlong <[email protected]> * Call the node API in dashboard-assistant plugin to generate parameters Refactor unit test code Signed-off-by: gaobinlong <[email protected]> * Fix test failure Signed-off-by: gaobinlong <[email protected]> * Revert the code format Signed-off-by: gaobinlong <[email protected]> * Remove some empty lines Signed-off-by: gaobinlong <[email protected]> --------- Signed-off-by: gaobinlong <[email protected]> (cherry picked from commit ec02b63) Co-authored-by: gaobinlong <[email protected]>
Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]>
…ct#887) Signed-off-by: Jackie Han <[email protected]>
Signed-off-by: Jackie Han <[email protected]>
jackiehanyang
requested review from
ohltyler,
kaituo,
amitgalitz,
sean-zheng-amazon,
dbwiddis,
owaiskazi19 and
joshpalis
as code owners
opensearch-trigger-bot
bot
added
documentation
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.
Description
Fix the following issues:
Issues Resolved
[List any issues this PR will resolve]
Check List
--signoffBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.