Migrate to current PyO3 version (#275)

tools/garaga_rs/Cargo.toml

@@ -26,14 +26,14 @@ wasm-bindgen-test = "0.3"
 
 # assumes python dependencies when not on wasm32-unknown-unknown
 [target.'cfg(not(target_arch = "wasm32"))'.dependencies]
-pyo3 = { version = "0.22", features = ["extension-module", "num-bigint"] }
+pyo3 = { version = "0.23", features = ["extension-module", "num-bigint"] }
 
 # assumes wasm dependencies when on wasm32-unknown-unknown
 [target.'cfg(target_arch = "wasm32")'.dependencies]
 wasm-bindgen = "0.2"
 
 [dependencies]
-pyo3 = { version = "0.22", features = ["extension-module", "num-bigint"], optional = true }
+pyo3 = { version = "0.23", features = ["extension-module", "num-bigint"], optional = true }
 wasm-bindgen = { version = "0.2", optional = true }
 num-bigint = "0.4"
 num-traits = "0.2"

tools/garaga_rs/src/python_bindings/ecip.rs

@@ -20,7 +20,10 @@ pub fn zk_ecip_hint(
     let v = ecip::core::zk_ecip_hint(list_values, list_scalars, curve_id)
         .map_err(PyErr::new::<pyo3::exceptions::PyValueError, _>)?;
 
-    let py_list = PyList::new_bound(py, v.into_iter().map(|x| PyList::new_bound(py, x)));
-
+    let inner_lists: Vec<_> = v
+        .into_iter()
+        .map(|x| PyList::new(py, x))
+        .collect::<PyResult<_>>()?;
+    let py_list = PyList::new(py, inner_lists)?;
     Ok(py_list.into())
 }

tools/garaga_rs/src/python_bindings/extf_mul.rs

@@ -53,6 +53,8 @@ where
         .into_iter()
         .map(|x| BigUint::from_bytes_be(&x.to_bytes_be()))
         .collect::<Vec<BigUint>>();
-    let py_tuple = PyTuple::new_bound(py, [PyList::new_bound(py, q), PyList::new_bound(py, r)]);
+    let q_list = PyList::new(py, q)?;
+    let r_list = PyList::new(py, r)?;
+    let py_tuple = PyTuple::new(py, [q_list, r_list])?;
     Ok(py_tuple.into())
 }

tools/garaga_rs/src/python_bindings/final_exp_witness.rs

@@ -20,6 +20,8 @@ pub fn get_final_exp_witness(
     let f_11: BigUint = py_list.get_item(11)?.extract()?;
     let f = [f_0, f_1, f_2, f_3, f_4, f_5, f_6, f_7, f_8, f_9, f_10, f_11];
     let (c, wi) = crate::pairing::final_exp_witness::get_final_exp_witness(curve_id, f);
-    let py_tuple = PyTuple::new_bound(py, [PyList::new_bound(py, c), PyList::new_bound(py, wi)]);
-    Ok(py_tuple.into())
+    let c_list = PyList::new(py, c)?;
+    let wi_list = PyList::new(py, wi)?;
+    let py_tuple = PyTuple::new(py, [c_list, wi_list])?;
+    Ok(Py::from(py_tuple))
 }

tools/garaga_rs/src/python_bindings/g2.rs

@@ -27,7 +27,7 @@ pub fn g2_add(
             Fq2::new(Fq::from(b_2), Fq::from(b_3)),
         );
         let c: G2Affine = (a + b).into();
-        let py_tuple = PyTuple::new_bound(
+        let py_tuple = PyTuple::new(
             py,
             [
                 BigUint::from(c.x.c0.into_bigint()),
@@ -36,7 +36,7 @@ pub fn g2_add(
                 BigUint::from(c.y.c1.into_bigint()),
             ],
         );
-        return Ok(py_tuple.into());
+        return Ok(py_tuple?.into_any().into());
     }
 
     if curve_id == CURVE_BLS12_381 {
@@ -50,7 +50,7 @@ pub fn g2_add(
             Fq2::new(Fq::from(b_2), Fq::from(b_3)),
         );
         let c: G2Affine = (a + b).into();
-        let py_tuple = PyTuple::new_bound(
+        let py_tuple = PyTuple::new(
             py,
             [
                 BigUint::from(c.x.c0.into_bigint()),
@@ -59,7 +59,7 @@ pub fn g2_add(
                 BigUint::from(c.y.c1.into_bigint()),
             ],
         );
-        return Ok(py_tuple.into());
+        return Ok(py_tuple?.into());
     }
 
     panic!("Curve ID {} not supported", curve_id);
@@ -85,7 +85,7 @@ pub fn g2_scalar_mul(
             Fq2::new(Fq::from(a_2), Fq::from(a_3)),
         );
         let c: G2Affine = a.mul_bigint(k.to_u64_digits()).into();
-        let py_tuple = PyTuple::new_bound(
+        let py_tuple = PyTuple::new(
             py,
             [
                 BigUint::from(c.x.c0.into_bigint()),
@@ -94,7 +94,7 @@ pub fn g2_scalar_mul(
                 BigUint::from(c.y.c1.into_bigint()),
             ],
         );
-        return Ok(py_tuple.into());
+        return Ok(py_tuple?.into());
     }
 
     if curve_id == CURVE_BLS12_381 {
@@ -104,7 +104,7 @@ pub fn g2_scalar_mul(
             Fq2::new(Fq::from(a_2), Fq::from(a_3)),
         );
         let c: G2Affine = a.mul_bigint(k.to_u64_digits()).into();
-        let py_tuple = PyTuple::new_bound(
+        let py_tuple = PyTuple::new(
             py,
             [
                 BigUint::from(c.x.c0.into_bigint()),
@@ -113,7 +113,7 @@ pub fn g2_scalar_mul(
                 BigUint::from(c.y.c1.into_bigint()),
             ],
         );
-        return Ok(py_tuple.into());
+        return Ok(py_tuple?.into());
     }
 
     panic!("Curve ID {} not supported", curve_id);

tools/garaga_rs/src/python_bindings/groth16_calldata.rs

@@ -35,6 +35,6 @@ pub fn get_groth16_calldata(
     )
     .map_err(PyErr::new::<pyo3::exceptions::PyValueError, _>)?;
 
-    let py_list = PyList::new_bound(py, result);
-    Ok(py_list.into())
+    let py_list = PyList::new(py, result);
+    Ok(py_list?.into())
 }

tools/garaga_rs/src/python_bindings/hades_permutation.rs

@@ -22,13 +22,12 @@ pub fn hades_permutation(
 
     PoseidonCairoStark252::hades_permutation(&mut state);
 
-    let py_tuple = PyTuple::new_bound(
+    let py_tuple = PyTuple::new(
         py,
         state.iter().map(|fe| {
             let fe_bytes = fe.to_bytes_be();
-            PyBytes::new_bound(py, &fe_bytes)
+            PyBytes::new(py, &fe_bytes)
         }),
     );
-
-    Ok(py_tuple.into())
+    Ok(py_tuple?.into())
 }

tools/garaga_rs/src/python_bindings/mpc_calldata.rs

@@ -20,6 +20,6 @@ pub fn mpc_calldata_builder(
         curve_id, &values1, n_fixed_g2, &values2,
     )
     .map_err(PyErr::new::<pyo3::exceptions::PyValueError, _>)?;
-    let py_list = PyList::new_bound(py, result);
-    Ok(py_list.into())
+    let py_list = PyList::new(py, result);
+    Ok(py_list?.into())
 }

tools/garaga_rs/src/python_bindings/msm.rs

@@ -30,6 +30,6 @@ pub fn msm_calldata_builder(
         risc0_mode,
     )
     .map_err(PyErr::new::<pyo3::exceptions::PyValueError, _>)?;
-    let py_list = PyList::new_bound(py, result);
-    Ok(py_list.into())
+    let py_list = PyList::new(py, result);
+    Ok(py_list?.into())
 }

tools/garaga_rs/src/python_bindings/pairing.rs

@@ -44,8 +44,8 @@ pub fn multi_pairing(
                 BigUint::from(v.c1.c2.c1.into_bigint()),
             ]
         }
-        let py_list = PyList::new_bound(py, to(c.0));
-        return Ok(py_list.into());
+        let py_list = PyList::new(py, to(c.0));
+        return Ok(py_list?.into());
     }
 
     if curve_id == CURVE_BLS12_381 {
@@ -84,8 +84,8 @@ pub fn multi_pairing(
                 BigUint::from(v.c1.c2.c1.into_bigint()),
             ]
         }
-        let py_list = PyList::new_bound(py, to(c.0));
-        return Ok(py_list.into());
+        let py_list = PyList::new(py, to(c.0));
+        return Ok(py_list?.into());
     }
 
     panic!("Curve ID {} not supported", curve_id);
@@ -135,8 +135,8 @@ pub fn multi_miller_loop(
                 BigUint::from(v.c1.c2.c1.into_bigint()),
             ]
         }
-        let py_list = PyList::new_bound(py, to(c.0));
-        return Ok(py_list.into());
+        let py_list = PyList::new(py, to(c.0));
+        return Ok(py_list?.into());
     }
 
     if curve_id == CURVE_BLS12_381 {
@@ -175,8 +175,8 @@ pub fn multi_miller_loop(
                 BigUint::from(v.c1.c2.c1.into_bigint()),
             ]
         }
-        let py_list = PyList::new_bound(py, to(c.0));
-        return Ok(py_list.into());
+        let py_list = PyList::new(py, to(c.0));
+        return Ok(py_list?.into());
     }
 
     panic!("Curve ID {} not supported", curve_id);