Fixed error with Rice 4.7

Author: ankane

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## 0.16.1 (unreleased)
 
 - Added `add_allowed_assignments` and `add_forbidden_assignments` methods to `CpModel`
+- Fixed error with Rice 4.7
 
 ## 0.16.0 (2025-06-19)
 

ext/or-tools/bin_packing.cpp

@@ -20,6 +20,10 @@ namespace Rice::detail {
   template<>
   class From_Ruby<KnapsackSolver::SolverType> {
   public:
+    From_Ruby() = default;
+
+    explicit From_Ruby(Arg* arg) : arg_(arg) { }
+
     Convertible is_convertible(VALUE value) { return Convertible::Cast; }
 
     KnapsackSolver::SolverType convert(VALUE x) {
@@ -30,6 +34,9 @@ namespace Rice::detail {
         throw std::runtime_error("Unknown solver type: " + s);
       }
     }
+
+  private:
+    Arg* arg_ = nullptr;
   };
 } // namespace Rice::detail
 

ext/or-tools/constraint.cpp

@@ -40,6 +40,10 @@ namespace Rice::detail {
   template<>
   class From_Ruby<LinearExpr> {
   public:
+    From_Ruby() = default;
+
+    explicit From_Ruby(Arg* arg) : arg_(arg) { }
+
     Convertible is_convertible(VALUE value) { return Convertible::Cast; }
 
     LinearExpr convert(VALUE v) {
@@ -63,6 +67,9 @@ namespace Rice::detail {
 
       return expr;
     }
+
+  private:
+    Arg* arg_ = nullptr;
   };
 } // namespace Rice::detail
 
@@ -409,7 +416,7 @@ void init_constraint(Rice::Module& m) {
         auto a = Array();
         auto assumptions = self.sufficient_assumptions_for_infeasibility();
         for (const auto& v : assumptions) {
-          a.push(v);
+          a.push(v, false);
         }
         return a;
       });

ext/or-tools/linear.cpp

@@ -26,6 +26,10 @@ namespace Rice::detail {
 
   template<>
   struct From_Ruby<MPSolver::OptimizationProblemType> {
+    From_Ruby() = default;
+
+    explicit From_Ruby(Arg* arg) : arg_(arg) { }
+
     Convertible is_convertible(VALUE value) { return Convertible::Cast; }
 
     static MPSolver::OptimizationProblemType convert(VALUE x) {
@@ -38,6 +42,9 @@ namespace Rice::detail {
         throw std::runtime_error("Unknown optimization problem type: " + s);
       }
     }
+
+  private:
+    Arg* arg_ = nullptr;
   };
 } // namespace Rice::detail
 

ext/or-tools/math_opt.cpp

@@ -25,6 +25,10 @@ namespace Rice::detail {
 
   template<>
   struct From_Ruby<SolverType> {
+    From_Ruby() = default;
+
+    explicit From_Ruby(Arg* arg) : arg_(arg) { }
+
     Convertible is_convertible(VALUE value) { return Convertible::Cast; }
 
     static SolverType convert(VALUE x) {
@@ -53,6 +57,9 @@ namespace Rice::detail {
         throw std::runtime_error("Unknown solver type: " + s);
       }
     }
+
+  private:
+    Arg* arg_ = nullptr;
   };
 } // namespace Rice::detail
 
@@ -61,7 +68,11 @@ void init_math_opt(Rice::Module& m) {
 
   Rice::define_class_under<Variable>(mathopt, "Variable")
     .define_method("id", &Variable::id)
-    .define_method("name", &Variable::name)
+    .define_method(
+      "name",
+      [](Variable& self) {
+        return std::string(self.name());
+      })
     .define_method(
       "_eql?",
       [](Variable& self, Variable &other) {

ext/or-tools/network_flows.cpp

@@ -47,7 +47,7 @@ void init_network_flows(Rice::Module& m) {
 
         Array ret;
         for (const auto& it : result) {
-          ret.push(it);
+          ret.push(it, false);
         }
         return ret;
       })
@@ -59,7 +59,7 @@ void init_network_flows(Rice::Module& m) {
 
         Array ret;
         for (const auto& it : result) {
-          ret.push(it);
+          ret.push(it, false);
         }
         return ret;
       });

ext/or-tools/routing.cpp

@@ -36,20 +36,34 @@ namespace Rice::detail {
   template<>
   class From_Ruby<RoutingNodeIndex> {
   public:
+    From_Ruby() = default;
+
+    explicit From_Ruby(Arg* arg) : arg_(arg) { }
+
     Convertible is_convertible(VALUE value) { return Convertible::Cast; }
 
     RoutingNodeIndex convert(VALUE x) {
       const RoutingNodeIndex index{From_Ruby<int>().convert(x)};
       return index;
     }
+
+  private:
+    Arg* arg_ = nullptr;
   };
 
   template<>
   class To_Ruby<RoutingNodeIndex> {
   public:
+    To_Ruby() = default;
+
+    explicit To_Ruby(Arg* arg) : arg_(arg) { }
+
     VALUE convert(RoutingNodeIndex const & x) {
       return To_Ruby<int>().convert(x.value());
     }
+
+  private:
+    Arg* arg_ = nullptr;
   };
 } // namespace Rice::detail
 
@@ -62,7 +76,7 @@ void init_routing(Rice::Module& m) {
   rb_cRoutingSearchParameters
     .define_method(
       "first_solution_strategy=",
-      [](RoutingSearchParameters& self, Symbol value) {
+      [](RoutingSearchParameters& self, Object value) {
         auto s = Symbol(value).str();
 
         FirstSolutionStrategy::Value v;
@@ -102,7 +116,7 @@ void init_routing(Rice::Module& m) {
       })
     .define_method(
       "local_search_metaheuristic=",
-      [](RoutingSearchParameters& self, Symbol value) {
+      [](RoutingSearchParameters& self, Object value) {
         auto s = Symbol(value).str();
 
         LocalSearchMetaheuristic::Value v;

or-tools.gemspec

@@ -16,5 +16,5 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = ">= 3.1"
 
-  spec.add_dependency "rice", ">= 4.5.0"
+  spec.add_dependency "rice", ">= 4.7"
 end