make riverlane arguments required

Author: BatmanAoD

crates/python/qcs_sdk/qpu/translation.pyi

@@ -143,7 +143,7 @@ class Riverlane:
     are specially authorized and are not generally available to the client.
     """
 
-    def __new__(cls, /, qeci_configuration_data: Optional[Dict[str, bytes]] = ...) -> Self: ...
+    def __new__(cls, qeci_configuration_data: Dict[str, bytes] = ..., qeci_max_nanoseconds_until_read_available = int) -> Self: ...
 
 @final
 class TranslationOptions:

crates/python/src/qpu/translation.rs

@@ -17,9 +17,6 @@ use rigetti_pyo3::{create_init_submodule, py_wrap_error, py_wrap_simple_enum, To
 
 use crate::client::PyQcsClient;
 
-// XXX is there a reasonable default?
-const DEFAULT_QECI_MAX_NANOS_READ: u64 = 8787;
-
 create_init_submodule! {
     classes: [
         PyTranslationOptions,
@@ -211,16 +208,15 @@ pub struct PyRiverlane(Riverlane);
 impl PyRiverlane {
     #[new]
     #[pyo3(
-        signature = (/, qeci_configuration_data = None, qeci_max_nanoseconds_until_read_available = None)
+        signature = (qeci_configuration_data, qeci_max_nanoseconds_until_read_available, /)
     )]
     fn __new__(
-        qeci_configuration_data: Option<HashMap<String, Vec<u8>>>,
-        qeci_max_nanoseconds_until_read_available: Option<u64>,
+        qeci_configuration_data: HashMap<String, Vec<u8>>,
+        qeci_max_nanoseconds_until_read_available: u64,
     ) -> PyResult<Self> {
         Ok(Self(Riverlane {
-            qeci_configuration_data: qeci_configuration_data.unwrap_or_default(),
-            qeci_max_nanoseconds_until_read_available: qeci_max_nanoseconds_until_read_available
-                .unwrap_or(DEFAULT_QECI_MAX_NANOS_READ),
+            qeci_configuration_data,
+            qeci_max_nanoseconds_until_read_available,
         }))
     }
 }