feat(data-access): geo-experiment auto-extend phases, post_extension insights types and extension config by ssilare-adobe · Pull Request #1972 · adobe/spacecat-shared · GitHub
Skip to content
This repository was archived by the owner on Oct 14, 2026. It is now read-only.

feat(data-access): geo-experiment auto-extend phases, post_extension insights types and extension config - #1972

Open
ssilare-adobe wants to merge 2 commits into
mainfrom
feat/geo-experiment-post-extension-insights-types
Open

ssilare-adobe wants to merge 2 commits into
mainfrom
feat/geo-experiment-post-extension-insights-types

Conversation

@ssilare-adobe

Copy link
Copy Markdown
Contributor

Summary

Data-model support for geo-experiment auto-extend (post window keeps running for up to 4 windows of 14 runs, same post DRS schedule).

  • INSIGHTS_TYPES: baseline, post_analysis, post_extension_{n} (window N ≥ 2 → post_extension_{N-1}); insightsTypeForWindow updated.
  • runRange on insightsList entries is now { min, max, taken: [ascending run numbers] } — taken is the post run numbers Mystique's selection actually used (gaps from missing / dropped runs allowed); min/max must equal first/last taken.
  • New phases: POST_EXTENSION_STARTED, POST_EXTENSION_ANALYSIS_STARTED, POST_EXTENSION_ANALYSIS_DONE, POST_EXTENSION_IMPACT_MEASUREMENT_STARTED, POST_EXTENSION_IMPACT_MEASUREMENT_DONE.
  • EXTENSION_CONFIG_ENV_VAR = 'EXPERIMENT_EXTENSION_CONFIG', METADATA_KEYS.EXTENSION, EXTENSION_CONFIG_KEYS, EXTENSION_MAX_WINDOWS = 4, EXTENSION_DEFAULT_WINDOW_SIZE = 14.
  • Typings (GeoExperimentRunRange, GeoExperimentExtension) and unit tests updated.

Breaking for writers of the previous extension type / {from,to} runRange — none in this package; consumers (api-service, engine) are updated in companion PRs.

Design: https://wiki.corp.adobe.com/pages/viewpage.action?pageId=4058181823

Test plan

  • npm test -w packages/spacecat-shared-data-access (3068 passing, geo-experiment.model.js 100%)
  • eslint on changed files

@github-actions

Copy link
Copy Markdown

This branch has not been deployed

No deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants