Skip to content

add tracking error logging #52

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add tracking error logging #52

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
04115f6
add tracking error logging
vanyakosmos Jul 10, 2025
73828f3
remove useless f-string
vanyakosmos Jul 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 15 additions & 15 deletions growthbook/growthbook.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
import json
import threading
import logging
import warnings

from abc import ABC, abstractmethod
from typing import Optional, Any, Set, Tuple, List, Dict, Callable
Expand Down Expand Up @@ -532,7 +533,6 @@ def _on_feature_update(self, features_data: Dict) -> None:
self._saved_groups = features_data["savedGroups"]

def load_features(self) -> None:

response = feature_repo.load_features(
self._api_host, self._client_key, self._decryption_key, self._cache_ttl
)
Expand Down Expand Up @@ -593,8 +593,8 @@ def startAutoRefresh(self):
def stopAutoRefresh(self):
feature_repo.stopAutoRefresh()

# @deprecated, use set_features
def setFeatures(self, features: dict) -> None:
warnings.warn("setFeatures is deprecated, use set_features instead", DeprecationWarning)
return self.set_features(features)

def set_features(self, features: dict) -> None:
Expand All @@ -612,23 +612,23 @@ def set_features(self, features: dict) -> None:
self._global_ctx.saved_groups = self._saved_groups
self.refresh_sticky_buckets()

# @deprecated, use get_features
def getFeatures(self) -> Dict[str, Feature]:
warnings.warn("getFeatures is deprecated, use get_features instead", DeprecationWarning)
return self.get_features()

def get_features(self) -> Dict[str, Feature]:
return self._features

# @deprecated, use set_attributes
def setAttributes(self, attributes: dict) -> None:
warnings.warn("setAttributes is deprecated, use set_attributes instead", DeprecationWarning)
return self.set_attributes(attributes)

def set_attributes(self, attributes: dict) -> None:
self._attributes = attributes
self.refresh_sticky_buckets()

# @deprecated, use get_attributes
def getAttributes(self) -> dict:
warnings.warn("getAttributes is deprecated, use get_attributes instead", DeprecationWarning)
return self.get_attributes()

def get_attributes(self) -> dict:
Expand All @@ -652,30 +652,30 @@ def destroy(self) -> None:
self._attributes.clear()
self._features.clear()

# @deprecated, use is_on
def isOn(self, key: str) -> bool:
warnings.warn("isOn is deprecated, use is_on instead", DeprecationWarning)
return self.is_on(key)

def is_on(self, key: str) -> bool:
return self.evalFeature(key).on
return self.eval_feature(key).on

# @deprecated, use is_off
def isOff(self, key: str) -> bool:
warnings.warn("isOff is deprecated, use is_off instead", DeprecationWarning)
return self.is_off(key)

def is_off(self, key: str) -> bool:
return self.evalFeature(key).off
return self.eval_feature(key).off

# @deprecated, use get_feature_value
def getFeatureValue(self, key: str, fallback):
warnings.warn("getFeatureValue is deprecated, use get_feature_value instead", DeprecationWarning)
return self.get_feature_value(key, fallback)

def get_feature_value(self, key: str, fallback):
res = self.evalFeature(key)
res = self.eval_feature(key)
return res.value if res.value is not None else fallback

# @deprecated, use eval_feature
def evalFeature(self, key: str) -> FeatureResult:
warnings.warn("evalFeature is deprecated, use eval_feature instead", DeprecationWarning)
return self.eval_feature(key)

def _ensure_fresh_features(self) -> None:
Expand Down Expand Up @@ -712,8 +712,8 @@ def eval_feature(self, key: str) -> FeatureResult:
tracking_cb=self._track
)

# @deprecated, use get_all_results
def getAllResults(self):
warnings.warn("getAllResults is deprecated, use get_all_results instead", DeprecationWarning)
return self.get_all_results()

def get_all_results(self):
Expand Down Expand Up @@ -766,8 +766,8 @@ def _track(self, experiment: Experiment, result: Result) -> None:
try:
self._trackingCallback(experiment=experiment, result=result)
self._tracked[key] = True
except Exception:
pass
except Exception as e:
logger.exception(e)

def _derive_sticky_bucket_identifier_attributes(self) -> List[str]:
attributes = set()
Expand Down