statsig-python-core 0.17.1rc2603201903__tar.gz → 0.17.2__tar.gz

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/Cargo.lock

@@ -2706,7 +2706,7 @@ dependencies = [
 
 [[package]]
 name = "sigstat-grpc"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 dependencies = [
  "async-trait",
  "chrono",
@@ -2792,7 +2792,7 @@ checksum = "a2eb9349b6444b326872e140eb1cf5e7c522154d69e7a0ffb0fb81c06b37543f"
 
 [[package]]
 name = "statsig-node"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 dependencies = [
  "async-trait",
  "napi",
@@ -2806,7 +2806,7 @@ dependencies = [
 
 [[package]]
 name = "statsig-pyo3"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 dependencies = [
  "async-trait",
  "lazy_static",
@@ -2820,7 +2820,7 @@ dependencies = [
 
 [[package]]
 name = "statsig-rust"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 dependencies = [
  "ahash",
  "arc-swap",
@@ -2866,7 +2866,7 @@ dependencies = [
 
 [[package]]
 name = "statsig_elixir"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 dependencies = [
  "async-trait",
  "parking_lot",
@@ -2879,7 +2879,7 @@ dependencies = [
 
 [[package]]
 name = "statsig_ffi"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 dependencies = [
  "async-trait",
  "cbindgen",

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/Cargo.toml

@@ -18,7 +18,7 @@ exclude = [
 [workspace.package]
 edition = "2021"
 license = "ISC"
-version = "0.17.1-rc.2603201903"
+version = "0.17.2"
 homepage = "https://statsig.com/"
 authors = ["Statsig", "Daniel Loomb <daniel@statsig.com>"]
 readme = "README.md"

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: statsig_python_core
-Version: 0.17.1rc2603201903
+Version: 0.17.2
 Classifier: Programming Language :: Rust
 Classifier: Programming Language :: Python :: Implementation :: CPython
 Classifier: Programming Language :: Python :: Implementation :: PyPy

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/py_src/statsig_python_core/statsig_python_core.pyi

@@ -298,9 +298,9 @@ class StatsigOptions:
     @output_log_level.setter
     def output_log_level(self, value: typing.Optional[builtins.str]) -> None: ...
     @property
-    def global_custom_fields(self) -> typing.Optional[builtins.dict[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]: ...
+    def global_custom_fields(self) -> typing.Optional[builtins.dict[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]: ...
     @global_custom_fields.setter
-    def global_custom_fields(self, value: typing.Optional[builtins.dict[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]) -> None: ...
+    def global_custom_fields(self, value: typing.Optional[builtins.dict[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]) -> None: ...
     @property
     def observability_client(self) -> typing.Optional[ObservabilityClientBase]: ...
     @observability_client.setter
@@ -345,7 +345,7 @@ class StatsigOptions:
     def experimental_flags(self) -> typing.Optional[builtins.set[builtins.str]]: ...
     @experimental_flags.setter
     def experimental_flags(self, value: typing.Optional[builtins.set[builtins.str]]) -> None: ...
-    def __new__(cls, specs_url: typing.Optional[builtins.str] = None, specs_sync_interval_ms: typing.Optional[builtins.int] = None, init_timeout_ms: typing.Optional[builtins.int] = None, log_event_url: typing.Optional[builtins.str] = None, disable_all_logging: typing.Optional[builtins.bool] = None, disable_network: typing.Optional[builtins.bool] = None, log_event_connection_reuse: typing.Optional[builtins.bool] = None, event_logging_flush_interval_ms: typing.Optional[builtins.int] = None, event_logging_max_queue_size: typing.Optional[builtins.int] = None, event_logging_max_pending_batch_queue_size: typing.Optional[builtins.int] = None, enable_id_lists: typing.Optional[builtins.bool] = None, enable_dcs_deltas: typing.Optional[builtins.bool] = None, wait_for_user_agent_init: typing.Optional[builtins.bool] = None, wait_for_country_lookup_init: typing.Optional[builtins.bool] = None, disable_country_lookup: typing.Optional[builtins.bool] = None, id_lists_url: typing.Optional[builtins.str] = None, download_id_list_file_api: typing.Optional[builtins.str] = None, id_lists_sync_interval_ms: typing.Optional[builtins.int] = None, fallback_to_statsig_api: typing.Optional[builtins.bool] = None, environment: typing.Optional[builtins.str] = None, service_name: typing.Optional[builtins.str] = None, output_log_level: typing.Optional[builtins.str] = None, global_custom_fields: typing.Optional[typing.Mapping[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]] = None, observability_client: typing.Optional[ObservabilityClientBase] = None, data_store: typing.Optional[DataStoreBase] = None, data_store_key_schema_version: typing.Optional[builtins.str] = None, persistent_storage: typing.Optional[PersistentStorageBaseClass] = None, config_compression_mode: typing.Optional[builtins.str] = None, proxy_config: typing.Optional[ProxyConfig] = None, output_logger_provider: typing.Optional[OutputLoggerProviderBase] = None, spec_adapter_configs: typing.Optional[list] = None, use_third_party_ua_parser: typing.Optional[builtins.bool] = None, disable_disk_access: typing.Optional[builtins.bool] = None, experimental_flags: typing.Optional[builtins.set[builtins.str]] = None) -> StatsigOptions: ...
+    def __new__(cls, specs_url: typing.Optional[builtins.str] = None, specs_sync_interval_ms: typing.Optional[builtins.int] = None, init_timeout_ms: typing.Optional[builtins.int] = None, log_event_url: typing.Optional[builtins.str] = None, disable_all_logging: typing.Optional[builtins.bool] = None, disable_network: typing.Optional[builtins.bool] = None, log_event_connection_reuse: typing.Optional[builtins.bool] = None, event_logging_flush_interval_ms: typing.Optional[builtins.int] = None, event_logging_max_queue_size: typing.Optional[builtins.int] = None, event_logging_max_pending_batch_queue_size: typing.Optional[builtins.int] = None, enable_id_lists: typing.Optional[builtins.bool] = None, enable_dcs_deltas: typing.Optional[builtins.bool] = None, wait_for_user_agent_init: typing.Optional[builtins.bool] = None, wait_for_country_lookup_init: typing.Optional[builtins.bool] = None, disable_country_lookup: typing.Optional[builtins.bool] = None, id_lists_url: typing.Optional[builtins.str] = None, download_id_list_file_api: typing.Optional[builtins.str] = None, id_lists_sync_interval_ms: typing.Optional[builtins.int] = None, fallback_to_statsig_api: typing.Optional[builtins.bool] = None, environment: typing.Optional[builtins.str] = None, service_name: typing.Optional[builtins.str] = None, output_log_level: typing.Optional[builtins.str] = None, global_custom_fields: typing.Optional[typing.Mapping[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]] = None, observability_client: typing.Optional[ObservabilityClientBase] = None, data_store: typing.Optional[DataStoreBase] = None, data_store_key_schema_version: typing.Optional[builtins.str] = None, persistent_storage: typing.Optional[PersistentStorageBaseClass] = None, config_compression_mode: typing.Optional[builtins.str] = None, proxy_config: typing.Optional[ProxyConfig] = None, output_logger_provider: typing.Optional[OutputLoggerProviderBase] = None, spec_adapter_configs: typing.Optional[list] = None, use_third_party_ua_parser: typing.Optional[builtins.bool] = None, disable_disk_access: typing.Optional[builtins.bool] = None, experimental_flags: typing.Optional[builtins.set[builtins.str]] = None) -> StatsigOptions: ...
 
 @typing.final
 class StatsigUser:
@@ -382,13 +382,13 @@ class StatsigUser:
     @custom_ids.setter
     def custom_ids(self, value: typing.Optional[builtins.dict[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float]]]) -> None: ...
     @property
-    def custom(self) -> typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]: ...
+    def custom(self) -> typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]]: ...
     @custom.setter
-    def custom(self, value: typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]) -> None: ...
+    def custom(self, value: typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]]) -> None: ...
     @property
-    def private_attributes(self) -> typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]: ...
+    def private_attributes(self) -> typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]]: ...
     @private_attributes.setter
-    def private_attributes(self, value: typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]) -> None: ...
+    def private_attributes(self, value: typing.Optional[builtins.dict[builtins.str, typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]]]) -> None: ...
     @property
     def statsig_environment(self) -> typing.Optional[builtins.dict[builtins.str, builtins.str]]: ...
     @statsig_environment.setter

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-pyo3/src/pyo_utils.rs

@@ -201,6 +201,10 @@ pub fn get_string_from_py_dict_throw_on_none(
 }
 
 pub fn py_any_to_value(value: &Bound<PyAny>) -> PyResult<Value> {
+    if value.is_none() {
+        return Ok(Value::Null);
+    }
+
     if let Ok(val) = value.extract::<String>() {
         return Ok(Value::String(val));
     }
@@ -250,6 +254,10 @@ pub fn py_any_to_value(value: &Bound<PyAny>) -> PyResult<Value> {
 
 /// order matters in this function, please don't change
 pub fn py_any_to_dynamic_value(value: &Bound<PyAny>) -> PyResult<DynamicValue> {
+    if value.is_none() {
+        return Ok(DynamicValue::new());
+    }
+
     if let Ok(val) = value.extract::<String>() {
         return Ok(DynamicValue::from(val));
     }
@@ -323,6 +331,10 @@ pub fn opt_py_dict_ref_to_hashmap(
 }
 
 fn py_any_ref_to_dynamic_value(value: &Bound<'_, PyAny>) -> Option<DynamicValue> {
+    if value.is_none() {
+        return Some(DynamicValue::new());
+    }
+
     if let Some(dv) = try_as_cow_str(value) {
         return Some(DynamicValue::from(dv.as_ref()));
     }

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-pyo3/src/valid_primitives_py.rs

@@ -41,7 +41,7 @@ pub enum ValidPrimitivesPy {
     Bool(bool),
     Float(f64),
     Int(i64),
-    Array(Vec<ValidArrayItemPy>),
+    Array(Vec<Option<ValidArrayItemPy>>),
     String(String),
     Dictionary(HashMap<String, Option<ValidPrimitivesPy>>),
 }
@@ -62,7 +62,7 @@ impl<'a, 'py> FromPyObject<'a, 'py> for ValidPrimitivesPy {
             return Ok(ValidPrimitivesPy::Float(f));
         }
 
-        if let Ok(s) = ob.extract::<Vec<ValidArrayItemPy>>() {
+        if let Ok(s) = ob.extract::<Vec<Option<ValidArrayItemPy>>>() {
             return Ok(ValidPrimitivesPy::Array(s));
         }
 
@@ -87,7 +87,7 @@ impl<'a, 'py> FromPyObject<'a, 'py> for ValidPrimitivesPy {
 impl PyStubType for ValidPrimitivesPy {
     fn type_output() -> TypeInfo {
         TypeInfo {
-            name: "typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]".to_string(),
+            name: "typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]".to_string(),
             source_module: None,
             import: HashSet::from(["builtins".into(), "typing".into()]),
             type_refs: HashMap::new(),
@@ -105,7 +105,10 @@ impl ValidPrimitivesPy {
             ValidPrimitivesPy::Array(a) => {
                 let mapped = a
                     .into_iter()
-                    .map(ValidArrayItemPy::into_dynamic_value)
+                    .map(|v| match v {
+                        Some(v) => v.into_dynamic_value(),
+                        None => DynamicValue::new(),
+                    })
                     .collect::<Vec<_>>();
                 dyn_value!(mapped)
             }
@@ -133,7 +136,7 @@ pub enum ValidPrimitivesPyRef<'a> {
     Bool(bool),
     Float(f64),
     Int(i64),
-    Array(Vec<ValidArrayItemPyRef<'a>>),
+    Array(Vec<Option<ValidArrayItemPyRef<'a>>>),
     String(&'a str),
     Dictionary(HashMap<String, Option<ValidPrimitivesPyRef<'a>>>),
 }
@@ -141,7 +144,7 @@ pub enum ValidPrimitivesPyRef<'a> {
 impl PyStubType for ValidPrimitivesPyRef<'_> {
     fn type_output() -> TypeInfo {
         TypeInfo {
-            name: "typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]".to_string(),
+            name: "typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]".to_string(),
             source_module: None,
             import: HashSet::from(["builtins".into(), "typing".into()]),
             type_refs: HashMap::new(),
@@ -170,7 +173,7 @@ impl<'a> ValidPrimitivesPyRef<'a> {
         }
 
         if let Some(ref value) = value.array_value {
-            let mut mapped: Vec<ValidArrayItemPyRef<'a>> = Vec::with_capacity(value.len());
+            let mut mapped: Vec<Option<ValidArrayItemPyRef<'a>>> = Vec::with_capacity(value.len());
             for v in value {
                 mapped.push(json_value_to_valid_array_item_py(&v.json_value));
             }
@@ -190,19 +193,20 @@ impl<'a> ValidPrimitivesPyRef<'a> {
     }
 }
 
-fn json_value_to_valid_array_item_py(value: &Value) -> ValidArrayItemPyRef<'_> {
+fn json_value_to_valid_array_item_py(value: &Value) -> Option<ValidArrayItemPyRef<'_>> {
     match value {
-        Value::String(v) => ValidArrayItemPyRef::String(v.as_str()),
+        Value::Null => None,
+        Value::String(v) => Some(ValidArrayItemPyRef::String(v.as_str())),
         Value::Number(v) => {
             if let Some(v) = v.as_i64() {
-                ValidArrayItemPyRef::Int(v)
+                Some(ValidArrayItemPyRef::Int(v))
             } else if let Some(v) = v.as_f64() {
-                ValidArrayItemPyRef::Float(v)
+                Some(ValidArrayItemPyRef::Float(v))
             } else {
-                ValidArrayItemPyRef::String("")
+                Some(ValidArrayItemPyRef::String(""))
             }
         }
-        Value::Bool(v) => ValidArrayItemPyRef::Bool(*v),
-        _ => ValidArrayItemPyRef::String(""),
+        Value::Bool(v) => Some(ValidArrayItemPyRef::Bool(*v)),
+        _ => Some(ValidArrayItemPyRef::String("")),
     }
 }

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-pyo3/tests/mock_scrapi.py

@@ -3,7 +3,6 @@ from werkzeug import Response, Request
 import threading
 import json
 import gzip
-import time
 
 
 class MockScrapi:

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-pyo3/tests/test_exposure_logging.py

@@ -183,7 +183,8 @@ def test_custom_event_with_typed_metadata(statsig_setup):
     assert event["metadata"]["an_int"] == 123
     assert event["metadata"]["a_float"] == 1.5
     assert event["metadata"]["a_bool"] is True
-    assert "a_none" not in event["metadata"] # we don't send None values
+    assert "a_none" in event["metadata"]
+    assert event["metadata"]["a_none"] is None
     assert event["metadata"]["an_object"] == {"nested_int": 7, "nested_str": "x"}
     assert event["metadata"]["an_array"] == [1, "a", False, {"k": "v"}]
 

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-pyo3/tests/test_shutdown.py

@@ -20,7 +20,7 @@ def test_cycling(httpserver: HTTPServer):
         "/v2/download_config_specs/secret-key.json", response=dcs_content, method="GET"
     )
 
-    def run_statsig(inner_mock_scrapi: MockScrapi):
+    def run_statsig(inner_mock_scrapi: MockScrapi, thread_id: int):
         options = StatsigOptions()
         options.specs_url = inner_mock_scrapi.url_for_endpoint(
             "/v2/download_config_specs"
@@ -28,21 +28,18 @@ def test_cycling(httpserver: HTTPServer):
         options.log_event_url = inner_mock_scrapi.url_for_endpoint("/v1/log_event")
         options.output_log_level = "none"
 
-        print("running statsig")
         statsig = Statsig("secret-key", options)
 
-        print("initializing statsig")
         statsig.initialize().wait(timeout=1)
-        print("initialized statsig")
         for i in range(1111):
-            statsig.check_gate(StatsigUser("user-{}".format(i)), "test_public")
-        print("shutting down statsig")
+            statsig.check_gate(StatsigUser(f"user-{thread_id}-{i}"), "test_public")
+        wait_ms(1)
         statsig.shutdown().wait(timeout=1)
 
     threads = []
 
-    for _ in range(3):
-        t = threading.Thread(target=run_statsig, args=(mock_scrapi,))
+    for j in range(3):
+        t = threading.Thread(target=run_statsig, args=(mock_scrapi, j))
         threads.append(t)
         t.start()
 

statsig_python_core-0.17.2/statsig-pyo3/tests/test_statsig_user_none_fields.py

@@ -0,0 +1,27 @@
+from statsig_python_core import StatsigUser
+
+
+def test_none_entries_in_constructor_dicts_are_preserved():
+    user = StatsigUser(
+        user_id="user_123",
+        custom={"keep": "value", "keep_none": None},
+        private_attributes={"secret": None},
+    )
+
+    assert user.custom == {"keep": "value", "keep_none": None}
+    assert user.private_attributes == {"secret": None}
+
+
+def test_none_entries_in_constructor_lists_and_nested_dicts_are_preserved():
+    user = StatsigUser(
+        user_id="user_123",
+        custom={
+            "list_with_none": ["value", None, 2],
+            "dict_with_none": {"inner": None, "keep": "value"},
+        },
+    )
+
+    assert user.custom == {
+        "list_with_none": ["value", None, 2],
+        "dict_with_none": {"inner": None, "keep": "value"},
+    }

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-rust/Cargo.toml

@@ -39,7 +39,7 @@ serde_json = { version = "1.0.143", features = [
 ] }
 serde_with = "3.4.0"
 sha2 = "0.10.8"
-sigstat-grpc = { path = "../statsig-grpc", version = "0.17.1-rc.2603201903", optional = true }
+sigstat-grpc = { path = "../statsig-grpc", version = "0.17.2", optional = true }
 simple_logger = { version = "5.0.0" }
 tempfile = "3.8.1"
 tokio = { version = "1.39.1", features = ["full"] }

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-rust/src/specs_adapter/statsig_customized_specs_adapter.rs

@@ -114,9 +114,8 @@ impl SpecsAdapter for StatsigCustomizedSpecsAdapter {
                 Err(e) => {
                     log_w!(
                         TAG,
-                        "Failed to initialize from {} adapter: {:?}",
+                        "Failed to initialize from {} adapter: {e}",
                         adapter.get_type_name(),
-                        e
                     );
                     // Carry on
                 }

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-rust/src/specs_response/proto_specs.rs

@@ -723,7 +723,7 @@ fn rules_from_pb(rules: Vec<pb::Rule>) -> Result<Vec<Rule>, StatsigErr> {
 
                 is_experiment_group: pb_rule.is_experiment_group,
 
-                sampling_rate: None,
+                sampling_rate: sampling_rate_from_pb(pb_rule.sampling_rate)?,
                 return_value: return_value_from_pb(pb_rule.return_value)?,
             };
 
@@ -732,6 +732,23 @@ fn rules_from_pb(rules: Vec<pb::Rule>) -> Result<Vec<Rule>, StatsigErr> {
         .collect::<Result<Vec<Rule>, StatsigErr>>()
 }
 
+fn sampling_rate_from_pb(sampling_rate: Option<f32>) -> Result<Option<u64>, StatsigErr> {
+    let Some(sampling_rate) = sampling_rate else {
+        return Ok(None);
+    };
+
+    if !sampling_rate.is_finite() || sampling_rate < 0.0 || sampling_rate.fract() != 0.0 {
+        return Err(StatsigErr::ProtobufParseError(
+            "proto::Rule".to_string(),
+            format!(
+                "Expected sampling rate to be a non-negative whole number, got {sampling_rate}"
+            ),
+        ));
+    }
+
+    Ok(Some(sampling_rate as u64))
+}
+
 fn return_value_from_pb(
     return_value: Option<pb::ReturnValue>,
 ) -> Result<DynamicReturnable, StatsigErr> {
@@ -870,3 +887,29 @@ fn make_proto_parse_error(tag: &str, message: &str) -> Result<(), StatsigErr> {
         message.to_string(),
     ))
 }
+
+#[cfg(test)]
+mod tests {
+    use super::{pb, rules_from_pb};
+
+    #[test]
+    fn rules_from_pb_preserves_sampling_rate() {
+        let rules = rules_from_pb(vec![pb::Rule {
+            name: "rule".to_string(),
+            pass_percentage: 100,
+            id: "rule-id".to_string(),
+            salt: None,
+            conditions: vec![],
+            id_type: None,
+            return_value: None,
+            group_name: None,
+            config_delegate: None,
+            is_experiment_group: None,
+            is_control_group: None,
+            sampling_rate: Some(201.0),
+        }])
+        .expect("protobuf rule should parse");
+
+        assert_eq!(rules[0].sampling_rate, Some(201));
+    }
+}

{statsig_python_core-0.17.1rc2603201903 → statsig_python_core-0.17.2}/statsig-rust/src/statsig_metadata.rs

@@ -10,7 +10,7 @@ lazy_static! {
     static ref STATSIG_METADATA: RwLock<StatsigMetadata> = RwLock::new(StatsigMetadata::new());
 }
 
-pub const SDK_VERSION: &str = "0.17.1-rc.2603201903";
+pub const SDK_VERSION: &str = "0.17.2";
 
 const TAG: &str = stringify!(StatsigMetadata);