optimizely-sdk 5.3.0__tar.gz → 5.4.0__tar.gz

{optimizely_sdk-5.3.0 → optimizely-sdk-5.4.0}/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Optimizely Python SDK Changelog
 
+## 5.4.0
+December 19th, 2025
+
+### New Features
+
+Holdout features:
+
+- Added support for Holdouts in Feature Experimentation, enabling global traffic exclusion from experiments for statistical confidence validation. ([#469](https://github.com/optimizely/python-sdk/pull/469))
+- Added holdout support for default decision service ([#467](https://github.com/optimizely/python-sdk/pull/467))
+- Project Config updated for Holdout ([#464](https://github.com/optimizely/python-sdk/pull/464))
+
+---
+
+### Bug Fixes
+- Resolved issues with Holdout impression event handling and notification delivery. ([#471](https://github.com/optimizely/python-sdk/pull/471))
+
+---
+
+
 ## 5.3.0
 November 13th, 2025
 

{optimizely_sdk-5.3.0/optimizely_sdk.egg-info → optimizely-sdk-5.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.1
 Name: optimizely-sdk
-Version: 5.3.0
+Version: 5.4.0
 Summary: Python SDK for Optimizely Feature Experimentation, Optimizely Full Stack (legacy), and Optimizely Rollouts.
 Home-page: https://github.com/optimizely/python-sdk
 Author: Optimizely
@@ -17,34 +17,8 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Description-Content-Type: text/markdown
-License-File: LICENSE
-Requires-Dist: jsonschema>=3.2.0
-Requires-Dist: pyrsistent>=0.16.0
-Requires-Dist: requests>=2.21
-Requires-Dist: idna>=2.10
-Requires-Dist: rpds-py<0.20.0; python_version < "3.11"
 Provides-Extra: test
-Requires-Dist: flake8>=4.0.1; extra == "test"
-Requires-Dist: python-coveralls; extra == "test"
-Requires-Dist: jsonschema>=3.2.0; extra == "test"
-Requires-Dist: rpds-py<0.20.0; python_version < "3.11" and extra == "test"
-Requires-Dist: requests>=2.21; extra == "test"
-Requires-Dist: pyrsistent>=0.16.0; extra == "test"
-Requires-Dist: idna>=2.10; extra == "test"
-Requires-Dist: pytest>=6.2.0; extra == "test"
-Requires-Dist: coverage; extra == "test"
-Requires-Dist: funcsigs>=0.4; extra == "test"
-Requires-Dist: pytest-cov; extra == "test"
-Dynamic: author
-Dynamic: author-email
-Dynamic: classifier
-Dynamic: description
-Dynamic: description-content-type
-Dynamic: home-page
-Dynamic: license-file
-Dynamic: provides-extra
-Dynamic: requires-dist
-Dynamic: summary
+License-File: LICENSE
 
 Optimizely Feature Experimentation is A/B testing and feature management for product development teams. Experiment in any application. Make every feature on your roadmap an opportunity to learn. Learn more at https://www.optimizely.com/products/experiment/feature-experimentation/ or see our documentation at https://docs.developers.optimizely.com/experimentation/v4.0.0-full-stack/docs/welcome. # Optimizely Python SDK
 
@@ -304,6 +278,25 @@ idna (BSD 3-Clause License https://github.com/kjd/idna/blob/master/LICENSE.md)
 - Swift - https://github.com/optimizely/swift-sdk
 # Optimizely Python SDK Changelog
 
+## 5.4.0
+December 19th, 2025
+
+### New Features
+
+Holdout features:
+
+- Added support for Holdouts in Feature Experimentation, enabling global traffic exclusion from experiments for statistical confidence validation. ([#469](https://github.com/optimizely/python-sdk/pull/469))
+- Added holdout support for default decision service ([#467](https://github.com/optimizely/python-sdk/pull/467))
+- Project Config updated for Holdout ([#464](https://github.com/optimizely/python-sdk/pull/464))
+
+---
+
+### Bug Fixes
+- Resolved issues with Holdout impression event handling and notification delivery. ([#471](https://github.com/optimizely/python-sdk/pull/471))
+
+---
+
+
 ## 5.3.0
 November 13th, 2025
 

{optimizely_sdk-5.3.0 → optimizely-sdk-5.4.0}/optimizely/bucketer.py

@@ -12,7 +12,7 @@
 # limitations under the License.
 
 from __future__ import annotations
-from typing import Optional, TYPE_CHECKING
+from typing import Optional, TYPE_CHECKING, cast
 import math
 from sys import version_info
 
@@ -28,7 +28,7 @@ else:
 if TYPE_CHECKING:
     # prevent circular dependenacy by skipping import at runtime
     from .project_config import ProjectConfig
-    from .entities import Experiment, Variation
+    from .entities import Experiment, Variation, Holdout
     from .helpers.types import TrafficAllocation
 
 
@@ -104,8 +104,8 @@ class Bucketer:
 
     def bucket(
         self, project_config: ProjectConfig,
-        experiment: Experiment, user_id: str, bucketing_id: str
-    ) -> tuple[Optional[Variation], list[str]]:
+        experiment: Experiment | Holdout, user_id: str, bucketing_id: str
+    ) -> tuple[Variation | None, list[str]]:
         """ For a given experiment and bucketing ID determines variation to be shown to user.
 
         Args:
@@ -125,14 +125,9 @@ class Bucketer:
             project_config.logger.debug(message)
             return None, []
 
-        if isinstance(experiment, dict):
-            # This is a holdout dictionary
-            experiment_key = experiment.get('key', '')
-            experiment_id = experiment.get('id', '')
-        else:
-            # This is an Experiment object
-            experiment_key = experiment.key
-            experiment_id = experiment.id
+        # Handle both Experiment and Holdout entities
+        experiment_key = experiment.key
+        experiment_id = experiment.id
 
         if not experiment_key or not experiment_key.strip():
             message = 'Invalid entity key provided for bucketing. Returning nil.'
@@ -141,14 +136,9 @@ class Bucketer:
 
         variation_id, decide_reasons = self.bucket_to_entity_id(project_config, experiment, user_id, bucketing_id)
         if variation_id:
-            if isinstance(experiment, dict):
-                # For holdouts, find the variation in the holdout's variations array
-                variations = experiment.get('variations', [])
-                variation = next((v for v in variations if v.get('id') == variation_id), None)
-            else:
-                # For experiments, use the existing method
-                variation = project_config.get_variation_from_id_by_experiment_id(experiment_id, variation_id)
-            return variation, decide_reasons
+            variation = project_config.get_variation_from_id_by_experiment_id(experiment_id, variation_id)
+            # Cast is safe here because experiments always use Variation entities, not VariationDict
+            return cast('Optional[Variation]', variation), decide_reasons
 
         # No variation found - log message for empty traffic range
         message = 'Bucketed into an empty traffic range. Returning nil.'
@@ -158,7 +148,7 @@ class Bucketer:
 
     def bucket_to_entity_id(
         self, project_config: ProjectConfig,
-        experiment: Experiment, user_id: str, bucketing_id: str
+        experiment: Experiment | Holdout, user_id: str, bucketing_id: str
     ) -> tuple[Optional[str], list[str]]:
         """
         For a given experiment and bucketing ID determines variation ID to be shown to user.
@@ -176,58 +166,52 @@ class Bucketer:
         if not experiment:
             return None, decide_reasons
 
-        # Handle both Experiment objects and holdout dictionaries
-        if isinstance(experiment, dict):
-            # This is a holdout dictionary - holdouts don't have groups
-            experiment_key = experiment.get('key', '')
-            experiment_id = experiment.get('id', '')
-            traffic_allocations = experiment.get('trafficAllocation', [])
-            has_cmab = False
-            group_policy = None
-        else:
-            # This is an Experiment object
-            experiment_key = experiment.key
-            experiment_id = experiment.id
-            traffic_allocations = experiment.trafficAllocation
-            has_cmab = bool(experiment.cmab)
+        # Handle both Experiment and Holdout entities
+        # Both entities have key, id, and trafficAllocation attributes
+        from . import entities
+
+        experiment_key = experiment.key
+        experiment_id = experiment.id
+        traffic_allocations = experiment.trafficAllocation
+
+        # Determine if experiment is in a mutually exclusive group
+        # Holdouts don't have groupId or groupPolicy - use isinstance for type narrowing
+        if isinstance(experiment, entities.Experiment):
             group_policy = getattr(experiment, 'groupPolicy', None)
+            if group_policy and group_policy in GROUP_POLICIES:
+                group = project_config.get_group(experiment.groupId)
 
-        # Determine if experiment is in a mutually exclusive group.
-        # This will not affect evaluation of rollout rules or holdouts.
-        if group_policy and group_policy in GROUP_POLICIES:
-            group = project_config.get_group(experiment.groupId)
+                if not group:
+                    return None, decide_reasons
 
-            if not group:
-                return None, decide_reasons
+                user_experiment_id = self.find_bucket(
+                    project_config, bucketing_id, experiment.groupId, group.trafficAllocation,
+                )
 
-            user_experiment_id = self.find_bucket(
-                project_config, bucketing_id, experiment.groupId, group.trafficAllocation,
-            )
+                if not user_experiment_id:
+                    message = f'User "{user_id}" is in no experiment.'
+                    project_config.logger.info(message)
+                    decide_reasons.append(message)
+                    return None, decide_reasons
 
-            if not user_experiment_id:
-                message = f'User "{user_id}" is in no experiment.'
-                project_config.logger.info(message)
-                decide_reasons.append(message)
-                return None, decide_reasons
+                if user_experiment_id != experiment_id:
+                    message = f'User "{user_id}" is not in experiment "{experiment_key}" of group {experiment.groupId}.'
+                    project_config.logger.info(message)
+                    decide_reasons.append(message)
+                    return None, decide_reasons
 
-            if user_experiment_id != experiment_id:
-                message = f'User "{user_id}" is not in experiment "{experiment_key}" of group {experiment.groupId}.'
+                message = f'User "{user_id}" is in experiment {experiment_key} of group {experiment.groupId}.'
                 project_config.logger.info(message)
                 decide_reasons.append(message)
-                return None, decide_reasons
-
-            message = f'User "{user_id}" is in experiment {experiment_key} of group {experiment.groupId}.'
-            project_config.logger.info(message)
-            decide_reasons.append(message)
-
-        if has_cmab:
-            if experiment.cmab:
-                traffic_allocations = [
-                    {
-                        "entityId": "$",
-                        "endOfRange": experiment.cmab['trafficAllocation']
-                    }
-                ]
+
+        # Holdouts don't have cmab - use isinstance for type narrowing
+        if isinstance(experiment, entities.Experiment) and experiment.cmab:
+            traffic_allocations = [
+                {
+                    "entityId": "$",
+                    "endOfRange": experiment.cmab['trafficAllocation']
+                }
+            ]
 
         # Bucket user if not in white-list and in group (if any)
         variation_id = self.find_bucket(project_config, bucketing_id,