Skip to content

Commit 276b0bc

Browse files
zhukaihanCopilot
andauthored
fix: redundant logic
Co-authored-by: Copilot <[email protected]>
1 parent d929b6e commit 276b0bc

File tree

2 files changed

+13
-18
lines changed

2 files changed

+13
-18
lines changed

src/amplitude_experiment/exposure/exposure_service.py

Lines changed: 11 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -30,24 +30,20 @@ def to_exposure_events(exposure: Exposure, ttl_millis: int) -> List[BaseEvent]:
3030
unset_props = {}
3131
flag_type = variant.metadata.get('flagType') if variant.metadata is not None else None
3232
if flag_type != FLAG_TYPE_MUTUAL_EXCLUSION_GROUP:
33-
if is_default:
34-
unset_props[f'[Experiment] {flag_key}'] = '-'
35-
else:
36-
if variant.key:
37-
set_props[f'[Experiment] {flag_key}'] = variant.key
38-
elif variant.value:
39-
set_props[f'[Experiment] {flag_key}'] = variant.value
33+
if variant.key:
34+
set_props[f'[Experiment] {flag_key}'] = variant.key
35+
elif variant.value:
36+
set_props[f'[Experiment] {flag_key}'] = variant.value
4037

4138
# Build event properties.
4239
event_properties = {}
43-
if not is_default:
44-
event_properties['[Experiment] Flag Key'] = flag_key
45-
if variant.key:
46-
event_properties['[Experiment] Variant'] = variant.key
47-
elif variant.value:
48-
event_properties['[Experiment] Variant'] = variant.value
49-
if variant.metadata:
50-
event_properties['metadata'] = variant.metadata
40+
event_properties['[Experiment] Flag Key'] = flag_key
41+
if variant.key:
42+
event_properties['[Experiment] Variant'] = variant.key
43+
elif variant.value:
44+
event_properties['[Experiment] Variant'] = variant.value
45+
if variant.metadata:
46+
event_properties['metadata'] = variant.metadata
5147

5248
# Build event.
5349
event = BaseEvent(

src/amplitude_experiment/local/client.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,8 @@ def __init__(self, api_key: str, config: LocalEvaluationConfig = None):
5252

5353
# Exposure service is always instantiated, using deployment key if no api key provided
5454
exposure_config = self.config.exposure_config
55-
if not exposure_config.api_key:
56-
exposure_config.api_key = api_key
57-
exposure_instance = Amplitude(exposure_config.api_key, exposure_config)
55+
api_key_to_use = exposure_config.api_key or api_key
56+
exposure_instance = Amplitude(api_key_to_use, exposure_config)
5857
self.exposure_service = ExposureService(exposure_instance, ExposureFilter(exposure_config.cache_capacity))
5958
self.logger = logging.getLogger("Amplitude")
6059
self.logger.addHandler(logging.StreamHandler())

0 commit comments

Comments
 (0)