removed unnecessary comments

optimizely · FarhanAnjum-opti · Nov 27, 2024 · Nov 4, 2024 · Nov 4, 2024 · Nov 6, 2024
commit 27ef846e6e3e2215775393cc3e75c8a590f39c17
diff --git a/optimizely/decision_service.py b/optimizely/decision_service.py
@@ -602,8 +602,8 @@ def get_variations_for_feature_list(
 
                         if decision_variation:
                             self.logger.debug(
-                                'User "{}" bucketed into experiment "{}" of feature "{}".'.format(
-                                    user_context.user_id, experiment.key, feature.key)
+                                f'User "{user_context.user_id}" '
+                                f'bucketed into experiment "{experiment.key}" of feature "{feature.key}".'
                             )
                             decision = Decision(experiment, decision_variation, enums.DecisionSources.FEATURE_TEST)
                             decisions.append((decision, feature_reasons))

diff --git a/optimizely/optimizely.py b/optimizely/optimizely.py
@@ -1291,16 +1291,9 @@ def _decide_for_keys(
             self.logger.debug('Provided decide options is not an array. Using default decide options.')
             merged_decide_options = self.default_decide_options
 
-        # enabled_flags_only = OptimizelyDecideOption.ENABLED_FLAGS_ONLY in merged_decide_options
-
         decisions: dict[str, OptimizelyDecision] = {}
         valid_keys = []
         decision_reasons_dict = {}
-        # for key in keys:
-        #     decision = self._decide(user_context, key, decide_options)
-        #     if enabled_flags_only and not decision.enabled:
-        #         continue
-        #     decisions[key] = decision
 
         project_config = self.config_manager.get_config()
         flags_without_forced_decision: list[entities.FeatureFlag] = []
@@ -1344,7 +1337,6 @@ def _decide_for_keys(
             flag_decisions[flag_key] = decision
             decision_reasons_dict[flag_key] += reasons
 
-        print(decision_reasons_dict)
         for key in valid_keys:
             flag_decision = flag_decisions[key]
             decision_reasons = decision_reasons_dict[key]

diff --git a/optimizely/user_profile.py b/optimizely/user_profile.py
@@ -126,7 +126,6 @@ def load_user_profile(self, reasons: Optional[list[str]] = [],
             if user_profile is None:
                 message = "Unable to get a user profile from the UserProfileService."
                 reasons.append(message)
-                # self.logger.info(message)
             else:
                 if 'user_id' in user_profile and 'experiment_bucket_map' in user_profile:
                     self.user_profile = UserProfile(
@@ -142,8 +141,6 @@ def load_user_profile(self, reasons: Optional[list[str]] = [],
             message = str(exception)
             reasons.append(message)
             self.logger.exception(f'Unable to retrieve user profile for user "{self.user_id}"as lookup failed.')
-            # Todo: add error handler
-            # error_handler.handle_error()
 
         if self.user_profile is None:
             self.user_profile = UserProfile(self.user_id, {})
@@ -173,4 +170,3 @@ def save_user_profile(self, error_handler: Optional[BaseErrorHandler] = None) ->
         except Exception as exception:
             self.logger.warning(f'Failed to save user profile of user "{self.user_profile.user_id}" '
                                 f'for exception:{exception}".')
-            # error_handler.handle_error(exception)