From 0369d8d08e07b68bc8b588b90d7dad462a7a3c49 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 15 Nov 2025 03:00:58 +0000
Subject: [PATCH 1/3] Increase tolerance in test_json_serialization to fix
 flaky test

Co-authored-by: hombit <1784493+hombit@users.noreply.github.com>
---
 light-curve/.gitignore                            | 1 +
 light-curve/tests/light_curve_ext/test_feature.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/light-curve/.gitignore b/light-curve/.gitignore
index b1423e1a..ec9798b6 100644
--- a/light-curve/.gitignore
+++ b/light-curve/.gitignore
@@ -8,3 +8,4 @@ dist/
 .tox/
 *.pyc
 target/
+venv/
diff --git a/light-curve/tests/light_curve_ext/test_feature.py b/light-curve/tests/light_curve_ext/test_feature.py
index 2730269e..92696fc9 100644
--- a/light-curve/tests/light_curve_ext/test_feature.py
+++ b/light-curve/tests/light_curve_ext/test_feature.py
@@ -431,7 +431,7 @@ def test_json_serialization(feature):
 
     from_to_json = lc.feature_from_json(feature.to_json())
     values_from_to_json = from_to_json(*data)
-    assert_allclose(values, values_from_to_json)
+    assert_allclose(values, values_from_to_json, rtol=1e-6)
 
 
 def test_json_deserialization():

From 74c9ab875a5a4af950fc3dfad8fb6a422b2498e4 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 15 Nov 2025 03:18:56 +0000
Subject: [PATCH 2/3] Use fixed random seed in gen_lc instead of relaxing
 tolerance

Co-authored-by: hombit <1784493+hombit@users.noreply.github.com>
---
 light-curve/tests/light_curve_ext/test_feature.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/light-curve/tests/light_curve_ext/test_feature.py b/light-curve/tests/light_curve_ext/test_feature.py
index 92696fc9..661c4a5a 100644
--- a/light-curve/tests/light_curve_ext/test_feature.py
+++ b/light-curve/tests/light_curve_ext/test_feature.py
@@ -426,12 +426,12 @@ def test_bazin_fit_precise(algo):
 @pytest.mark.parametrize("feature", gen_feature_evaluators(parametric_variants=5, rng=0))
 def test_json_serialization(feature):
     n_obs = 128
-    data = gen_lc(n_obs)
+    data = gen_lc(n_obs, rng=0)
     values = feature(*data)
 
     from_to_json = lc.feature_from_json(feature.to_json())
     values_from_to_json = from_to_json(*data)
-    assert_allclose(values, values_from_to_json, rtol=1e-6)
+    assert_allclose(values, values_from_to_json)
 
 
 def test_json_deserialization():

From 9837bda4d3b31b8790a06aa8638a45576e11036e Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 15 Nov 2025 22:26:20 +0000
Subject: [PATCH 3/3] Remove venv/ from light-curve/.gitignore as it's already
 in root .gitignore

Co-authored-by: hombit <1784493+hombit@users.noreply.github.com>
---
 light-curve/.gitignore | 1 -
 1 file changed, 1 deletion(-)

diff --git a/light-curve/.gitignore b/light-curve/.gitignore
index ec9798b6..b1423e1a 100644
--- a/light-curve/.gitignore
+++ b/light-curve/.gitignore
@@ -8,4 +8,3 @@ dist/
 .tox/
 *.pyc
 target/
-venv/