recovered regr_smlp/data/smlp_toy_num_resp_noknobs.csv and regr_smlp/specs/smlp_toy_num_resp_noknobs_verify.spec#151
recovered regr_smlp/data/smlp_toy_num_resp_noknobs.csv and regr_smlp/specs/smlp_toy_num_resp_noknobs_verify.spec#151zurabksmlp wants to merge 20 commits into
Conversation
…specs/smlp_toy_num_resp_noknobs_verify.spec
mdmitry1
left a comment
There was a problem hiding this comment.
- There are many tests failed due to missing data:
echo n | env CUDA_VISIBLE_DEVICES=-1 python3.11 ./smlp_regr.py -t all -w 8 -def n -g |& tee noknobs_tests_recovery.smlp_regr.log
grep Failed noknobs_tests_recovery.smlp_regr.log
Test 73 Failed:
Test 74 Failed:
Test 75 Failed:
Test 139 Failed:
Test 162 Failed:
Test 163 Failed:
Test 184 Failed:
Test 185 Failed:
Test 186 Failed:
- Master is not updated
2.1 Missing files in master
grep "File master" noknobs_tests_recovery.smlp_regr.log | awk '{print $3}' | cut -d_ -f1 | sort -uTest144
Test57
Test61
Test62
test65
Test65
Test66
test67
Test67
Test68
test71
Test71
Test72
test76
Test76
test78
Test78
Test842.2 Diff(s) due smlp package structure change.
For example:
cat test76_model_smlp_model_term.json | tr ' ' '\012' | sort > test76_model_smlp_model_term_sorted.json
cat ../master/test76_model_smlp_model_term.json | tr ' ' '\012' | sort > test76_model_smlp_model_term_sorted_master.json
diff test76_model_smlp_model_term_sorted*.json
215,216c215,216
< <smlp.core.libsmlp.term2
< <smlp.core.libsmlp.term2
---
> <smlp.libsmlp.term2
> <smlp.libsmlp.term2
My results: code.tar.gz
Version validated:
pip show smlptech | grep VersionVersion: 1.2.0
|
Checking this on my pytest branch we go from to Warnings removed concern path
|
|
Results after 0459981 checkin. cd $(git rev-parse --show-toplevel)/tests/smlp_regression
./run_smlp_regression
grep -c Failed run_smlp_regression.log
grep "File master" run_smlp_regression.log | awk '{print $3}' | cut -d_ -f1 | sort -u
|
|
Now even better: The 3 tests now failing that are new in wrt. before are 57, 65 and 66, due to DetailsTest57: --- ../master/Test57_smlp_toy_num_resp_noknobs_verify_results.json 2026-06-01 14:02:21.048730624 +0200
+++ /tmp/pytest-of-kane/pytest-9/popen-gw13/test3/Test57_smlp_toy_num_resp_noknobs_verify_results.json 2026-06-01 14:03:44.418788290 +0200
@@ -4,10 +4,10 @@
"assertion_status": "FAIL",
"counter_example": {
"x0": 0.0,
- "x1": 1.0,
+ "x1": 4.0,
"y1": 9.0,
- "x2": 7.0,
- "y2": 5.0
+ "x2": 6.000000067055225,
+ "y2": 9.0
},
"assertion_feasible": false
},Test65 --- ../master/Test65_smlp_toy_num_resp_noknobs_verify_results.json 2026-06-01 14:02:21.054730700 +0200
+++ /tmp/pytest-of-kane/pytest-9/popen-gw6/test3/Test65_smlp_toy_num_resp_noknobs_verify_results.json 2026-06-01 14:03:49.898857820 +0200
@@ -10,7 +10,7 @@
"assertion_status": "FAIL",
"counter_example": {
"x0": 0.0,
- "x1": 7.0,
+ "x1": 4.0,
"y1": 9.0,
"x2": 6.000000067055225,
"y2": 9.0Test66 --- ../master/Test66_test65_model_trace.csv 2026-06-01 14:02:21.054730700 +0200
+++ /tmp/pytest-of-kane/pytest-9/popen-gw6/test4/Test66_test65_model_trace.csv 2026-06-01 14:04:04.129038377 +0200
@@ -1,9 +1,9 @@
stage,solver,x0,x1,x2,y1,y2
interface_consistency,sat,0,7,3
-model_consistency,sat,0,1,805306377/134217728,5,5
-witness_consistency,sat,0,1,805306377/134217728,5,5
-witness_consistency,sat,0,1,805306377/134217728,5,5
-ca,sat,0,1,6,5,5
+model_consistency,sat,0,7,805306377/134217728,9,9
+witness_consistency,sat,0,7,805306377/134217728,9,9
+witness_consistency,sat,0,7,805306377/134217728,9,9
+ca,sat,1,1,27/4,9,5
ce,unsat
-ca,sat,0,7,671088649/134217728,5,9
-ce,sat,0,7,805306377/134217728,9,9
+ca,sat,0,1,1140850697/201326592,5,5
+ce,sat,0,4,805306377/134217728,9,9
--- ../master/Test66_test65_model_verify_results.json 2026-06-01 14:02:21.054730700 +0200
+++ /tmp/pytest-of-kane/pytest-9/popen-gw6/test4/Test66_test65_model_verify_results.json 2026-06-01 14:04:04.133038427 +0200
@@ -10,7 +10,7 @@
"assertion_status": "FAIL",
"counter_example": {
"x0": 0.0,
- "x1": 7.0,
+ "x1": 4.0,
"y1": 9.0,
"x2": 6.000000067055225,
"y2": 9.0 |
…asters and models
|
Reproduced @fbrausse results for Test57:
|
Results after commit 88cdb5f Mon Jun 1 23:19:50 2026 +0300All inputs are in placeMissing master filesDiffsResults match exactly on two machines
SMLP packages |
|
Filtering out one more expected-to-fail test (229) as well as the three tests 145, 146 and 205 (for the reason mentioned in #149), pytest now arrives at |

Recovered missing data and spec files for the "noknobs" tests:
57,61,62,65,66,67,68,71,72,73,74,75,76,78,84,139,144,162,163,184,185,186
All tests use the same data and spec files:
data:
regr_smlp/data/smlp_toy_num_resp_noknobs
specL
regr_smlp/specs/smlp_toy_num_resp_noknobs_verify.spec
These data and spec files are obtained simply by renaming variables: : x --> x0, p1 --> x1, p2 --> x3 irrespectively n:
regr_smlp/data/smlp_toy_num_resp_mult.csv
regr_smlp/specs/smlp_toy_num_resp_mult_verify.spec
I have checked these changes on Test57 and have a good confidence that the recovered files are as intended.
This addresses part of missing input files issue #147