public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
* [rpms/python-pyscipopt] rawhide: Fix `test_markDoNotAggrVar_and_getStatus`
@ 2026-07-18 8:25 Benjamin A. Beasley
0 siblings, 0 replies; only message in thread
From: Benjamin A. Beasley @ 2026-07-18 8:25 UTC (permalink / raw)
To: git-commits
A new commit has been pushed.
Repo : rpms/python-pyscipopt
Branch : rawhide
Commit : 44b305e28b86fa562c8f5aee375ad3bb68ac02f1
Author : Benjamin A. Beasley <code@musicinmybrain.net>
Date : 2026-07-18T09:21:51+01:00
Stats : +76/-0 in 2 file(s)
URL : https://src.fedoraproject.org/rpms/python-pyscipopt/c/44b305e28b86fa562c8f5aee375ad3bb68ac02f1?branch=rawhide
Log:
Fix `test_markDoNotAggrVar_and_getStatus`
---
diff --git a/0001-Make-test_markDoNotAggrVar_and_getStatus-robust-to-S.patch b/0001-Make-test_markDoNotAggrVar_and_getStatus-robust-to-S.patch
new file mode 100644
index 0000000..2c736af
--- /dev/null
+++ b/0001-Make-test_markDoNotAggrVar_and_getStatus-robust-to-S.patch
@@ -0,0 +1,71 @@
+From f84585d4c4cb321da2bb5cde567bc5e433b8e848 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Jo=C3=A3o=20Dion=C3=ADsio?=
+ <57299939+Joao-Dionisio@users.noreply.github.com>
+Date: Thu, 25 Jun 2026 17:47:34 +0100
+Subject: [PATCH] Make test_markDoNotAggrVar_and_getStatus robust to SCIP
+ presolve changes (#1230)
+
+---
+ CHANGELOG.md | 1 +
+ tests/test_vars.py | 25 +++++++++++--------------
+ 2 files changed, 12 insertions(+), 14 deletions(-)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index 00173c5..47768b0 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -3,6 +3,7 @@
+ ## Unreleased
+ ### Added
+ ### Fixed
++- Made `test_markDoNotAggrVar_and_getStatus` robust to SCIP presolve changes by discovering the aggregated/multi-aggregated variables instead of hardcoding them
+ ### Changed
+ ### Removed
+
+diff --git a/tests/test_vars.py b/tests/test_vars.py
+index 43f7158..b32acfd 100644
+--- a/tests/test_vars.py
++++ b/tests/test_vars.py
+@@ -142,28 +142,25 @@ def test_markDoNotAggrVar_and_getStatus():
+ model.addCons(y - 2*x == 0)
+ model.addCons(x + z + w == 10)
+ model.addCons(x*y*z >= 21) # to prevent presolve from removing all variables
+- model.presolve()
+
+- assert z.getStatus() == "ORIGINAL"
+- assert model.getTransformedVar(z).getStatus() == "AGGREGATED"
+- assert model.getTransformedVar(w).getStatus() == "MULTAGGR"
++ variables = (x, y, z, w)
++ model.presolve()
+
+- assert model.getNVars(True) == 1
++ multaggr = [v for v in variables if model.getTransformedVar(v).getStatus() == "MULTAGGR"]
++ aggregated = [v for v in variables if model.getTransformedVar(v).getStatus() == "AGGREGATED"]
++ assert multaggr, "presolve no longer multi-aggregates; update the test model"
++ assert aggregated, "presolve no longer aggregates; update the test model"
++ assert multaggr[0].getStatus() == "ORIGINAL"
+
+ model.freeTransform()
+- model.markDoNotMultaggrVar(w)
++ model.markDoNotMultaggrVar(multaggr[0])
+ model.presolve()
+-
+- assert model.getTransformedVar(w).getStatus() != "MULTAGGR"
+- assert model.getNVars(True) == 3
++ assert model.getTransformedVar(multaggr[0]).getStatus() != "MULTAGGR"
+
+ model.freeTransform()
+- model.markDoNotAggrVar(y)
++ model.markDoNotAggrVar(aggregated[0])
+ model.presolve()
+- assert model.getTransformedVar(z).getStatus() != "AGGREGATED"
+- assert model.getNVars(True) == 4
+-
+- assert x.getStatus() == "ORIGINAL"
++ assert model.getTransformedVar(aggregated[0]).getStatus() != "AGGREGATED"
+
+
+ def test_isIntegral():
+--
+2.55.0
+
diff --git a/python-pyscipopt.spec b/python-pyscipopt.spec
index fb0234d..7a8c659 100644
--- a/python-pyscipopt.spec
+++ b/python-pyscipopt.spec
@@ -10,6 +10,11 @@ License: MIT AND WTFNMFPL
URL: https://github.com/scipopt/PySCIPOpt
Source: %{url}/archive/v%{version}/PySCIPOpt-%{version}.tar.gz
+# Fix `test_markDoNotAggrVar_and_getStatus`
+# https://github.com/scipopt/PySCIPOpt/pull/1230
+# Cherry-picked on v6.2.1
+Patch: 0001-Make-test_markDoNotAggrVar_and_getStatus-robust-to-S.patch
+
BuildSystem: pyproject
BuildOption(install): --assert-license pyscipopt
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2026-07-18 8:25 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-07-18 8:25 [rpms/python-pyscipopt] rawhide: Fix `test_markDoNotAggrVar_and_getStatus` Benjamin A. Beasley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox