lib.rs 43.3 KB
Newer Older
1
// SPDX-FileCopyrightText: Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2
3
// SPDX-License-Identifier: Apache-2.0

4
use dynamo_llm::local_model::LocalModel;
5
use dynamo_runtime::distributed::{DiscoveryBackend, DistributedConfig, RequestPlaneMode};
6
use dynamo_runtime::storage::kv;
7
8
use futures::StreamExt;
use once_cell::sync::OnceCell;
9
use pyo3::IntoPyObjectExt;
10
use pyo3::exceptions::PyStopAsyncIteration;
Richard Huo's avatar
Richard Huo committed
11
use pyo3::types::PyCapsule;
12
use pyo3::types::{PyDict, PyString};
13
14
use pyo3::{exceptions::PyException, prelude::*};
use rs::pipeline::network::Ingress;
Richard Huo's avatar
Richard Huo committed
15
use std::ffi::CString;
16
use std::fs;
17
use std::path::PathBuf;
Richard Huo's avatar
Richard Huo committed
18
19
20
21
use std::{
    fmt::Display,
    sync::{Arc, Weak},
};
22
use tokio::sync::Mutex;
23
use tracing::Instrument;
24

25
use dynamo_runtime::config;
26
use dynamo_runtime::config::environment_names::logging::otlp as env_otlp;
Neelay Shah's avatar
Neelay Shah committed
27
use dynamo_runtime::{
Ryan Olson's avatar
Ryan Olson committed
28
    self as rs, logging,
29
    pipeline::{
30
        AsyncEngineContextProvider, EngineStream, ManyOut, SingleIn, context::Context as RsContext,
31
        network::egress::push_router::RouterMode as RsRouterMode,
32
    },
33
    protocols::annotated::Annotated as RsAnnotated,
34
    traits::DistributedRuntimeProvider,
35
36
};

37
38
use dynamo_kv_router::config::KvRouterConfig;
use dynamo_llm::entrypoint::RouterConfig;
Neelay Shah's avatar
Neelay Shah committed
39
use dynamo_llm::{self as llm_rs};
40

41
use crate::llm::local_model::ModelRuntimeConfig;
42
use crate::llm::preprocessor::{MediaDecoder, MediaFetcher};
43

44
45
46
47
48
49
#[pyclass(eq, eq_int)]
#[derive(Clone, Debug, PartialEq)]
pub enum RouterMode {
    RoundRobin,
    Random,
    KV,
50
51
52
    /// Direct routing - reads worker ID from each request's routing hints.
    /// Used when an external orchestrator (e.g., EPP) handles worker selection.
    Direct,
53
54
55
56
57
58
59
60
}

impl From<RouterMode> for RsRouterMode {
    fn from(mode: RouterMode) -> Self {
        match mode {
            RouterMode::RoundRobin => Self::RoundRobin,
            RouterMode::Random => Self::Random,
            RouterMode::KV => Self::KV,
61
            RouterMode::Direct => Self::Direct,
62
63
64
        }
    }
}
65

66
mod context;
67
mod engine;
68
mod http;
69
mod kserve_grpc;
70
mod llm;
71
mod parsers;
72
mod planner;
73
mod prometheus_metrics;
74
75
76
77
78
79
80

type JsonServerStreamingIngress =
    Ingress<SingleIn<serde_json::Value>, ManyOut<RsAnnotated<serde_json::Value>>>;

static INIT: OnceCell<()> = OnceCell::new();

const DEFAULT_ANNOTATED_SETTING: Option<bool> = Some(true);
81
82
const SKIP_PYTHON_LOG_INIT_ENV: &str = "DYNAMO_SKIP_PYTHON_LOG_INIT";

83
84
// Helper to get appropriate span for instrumentation - always emit spans
fn get_span_for_context(context: &context::Context, operation: &str) -> tracing::Span {
85
86
87
88
89
90
    logging::make_client_request_span(
        operation,
        context.inner().id(),
        context.trace_context(),
        None,
    )
91
92
93
94
95
96
97
98
}

// Helper to create span for direct method with instance_id
fn get_span_for_direct_context(
    context: &context::Context,
    operation: &str,
    instance_id: &str,
) -> tracing::Span {
99
100
101
102
103
104
    logging::make_client_request_span(
        operation,
        context.inner().id(),
        context.trace_context(),
        Some(instance_id),
    )
105
106
}

107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
// Helper to create request context with proper linking and cancellation handling
fn create_request_context(
    request: serde_json::Value,
    parent_ctx: &Option<context::Context>,
) -> RsContext<serde_json::Value> {
    match parent_ctx {
        // If there is a parent context, link the request as a child context of it
        Some(parent_ctx) => {
            let child_ctx = RsContext::with_id(request, parent_ctx.inner().id().to_string());
            parent_ctx.inner().link_child(child_ctx.context());
            if parent_ctx.inner().is_stopped() || parent_ctx.inner().is_killed() {
                // Let the server handle the cancellation for now since not all backends are
                // properly handling request exceptions
                // TODO: (DIS-830) Return an error if context is cancelled
                child_ctx.context().stop_generating();
            }
            child_ctx
        }
        // Otherwise if there is no parent context, use the request as-is
        _ => request.into(),
    }
}

130
131
132
133
134
/// A Python module implemented in Rust. The name of this function must match
/// the `lib.name` setting in the `Cargo.toml`, else Python will not be able to
/// import the module.
#[pymodule]
fn _core(m: &Bound<'_, PyModule>) -> PyResult<()> {
135
    // Initialize logging early unless OTEL export is enabled (which requires tokio runtime)
136
    if config::env_is_truthy(env_otlp::OTEL_EXPORT_ENABLED) {
137
        eprintln!(
138
            "Warning: OTEL_EXPORT_ENABLED detected. Logging initialization deferred until runtime is available. Early logs may be dropped."
139
        );
140
    } else if std::env::var_os(SKIP_PYTHON_LOG_INIT_ENV).is_none() {
141
142
143
        rs::logging::init();
    }

Yan Ru Pei's avatar
Yan Ru Pei committed
144
    m.add_function(wrap_pyfunction!(llm::kv::compute_block_hash_for_seq_py, m)?)?;
145
    m.add_function(wrap_pyfunction!(lora_name_to_id, m)?)?;
Ryan Olson's avatar
Ryan Olson committed
146
    m.add_function(wrap_pyfunction!(log_message, m)?)?;
147
148
149
    m.add_function(wrap_pyfunction!(register_model, m)?)?;
    m.add_function(wrap_pyfunction!(unregister_model, m)?)?;
    m.add_function(wrap_pyfunction!(fetch_model, m)?)?;
150
    m.add_function(wrap_pyfunction!(run_kv_indexer, m)?)?;
151
    m.add_function(wrap_pyfunction!(llm::entrypoint::make_engine, m)?)?;
152
    m.add_function(wrap_pyfunction!(llm::replay::run_mocker_trace_replay, m)?)?;
153
    m.add_function(wrap_pyfunction!(
154
        llm::replay::run_mocker_synthetic_trace_replay,
155
156
        m
    )?)?;
157
    m.add_function(wrap_pyfunction!(llm::entrypoint::run_input, m)?)?;
158

159
160
    m.add_class::<DistributedRuntime>()?;
    m.add_class::<Endpoint>()?;
161
    m.add_class::<ModelCardInstanceId>()?;
162
163
    m.add_class::<Client>()?;
    m.add_class::<AsyncResponseStream>()?;
164
165
166
    m.add_class::<llm::entrypoint::EntrypointArgs>()?;
    m.add_class::<llm::entrypoint::EngineConfig>()?;
    m.add_class::<llm::entrypoint::EngineType>()?;
167
168
    m.add_class::<llm::entrypoint::RouterConfig>()?;
    m.add_class::<llm::entrypoint::KvRouterConfig>()?;
169
170
171
    m.add_class::<llm::replay::ReasoningConfig>()?;
    m.add_class::<llm::replay::SglangArgs>()?;
    m.add_class::<llm::replay::MockEngineArgs>()?;
172
    m.add_class::<llm::kv::WorkerMetricsPublisher>()?;
173
    m.add_class::<llm::model_card::ModelDeploymentCard>()?; // Internal: only in _internal, not public API
174
    m.add_class::<llm::local_model::ModelRuntimeConfig>()?;
175
176
    m.add_class::<llm::preprocessor::MediaDecoder>()?;
    m.add_class::<llm::preprocessor::MediaFetcher>()?;
177
    m.add_class::<llm::kv::OverlapScores>()?;
178
    m.add_class::<llm::kv::KvEventPublisher>()?;
Yan Ru Pei's avatar
Yan Ru Pei committed
179
    m.add_class::<llm::kv::RadixTree>()?;
180
181
    m.add_class::<llm::fpm::FpmEventRelay>()?;
    m.add_class::<llm::fpm::FpmEventSubscriber>()?;
182
    m.add_class::<llm::lora::LoRADownloader>()?;
183
184
    m.add_class::<http::HttpService>()?;
    m.add_class::<http::HttpAsyncEngine>()?;
185
    m.add_class::<context::Context>()?;
186
    m.add_class::<ModelType>()?;
187
    m.add_class::<ModelInput>()?;
188
    m.add_class::<llm::kv::KvRouter>()?;
189
    m.add_class::<RouterMode>()?;
190
    m.add_class::<kserve_grpc::KserveGrpcService>()?;
191
    m.add("__version__", env!("CARGO_PKG_VERSION"))?;
192
193
194
    m.add_class::<planner::VirtualConnectorCoordinator>()?;
    m.add_class::<planner::VirtualConnectorClient>()?;
    m.add_class::<planner::PlannerDecision>()?;
195
196

    engine::add_to_module(m)?;
197
    parsers::add_to_module(m)?;
198

199
    m.add_class::<prometheus_metrics::RuntimeMetrics>()?;
200
201
202
203
    let prometheus_metrics = PyModule::new(m.py(), "prometheus_metrics")?;
    prometheus_metrics::add_to_module(&prometheus_metrics)?;
    m.add_submodule(&prometheus_metrics)?;

204
205
206
207
208
209
210
211
212
213
    Ok(())
}

pub fn to_pyerr<E>(err: E) -> PyErr
where
    E: Display,
{
    PyException::new_err(format!("{}", err))
}

214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
fn kv_indexer_to_pyerr(err: anyhow::Error) -> PyErr {
    #[cfg(feature = "kv-indexer")]
    if let Some(clap_error) = err.downcast_ref::<clap::Error>() {
        let _ = clap_error.print();
        return pyo3::exceptions::PySystemExit::new_err(clap_error.exit_code());
    }

    to_pyerr(err)
}

#[pyfunction(name = "run_kv_indexer")]
#[pyo3(signature = (argv=None))]
fn run_kv_indexer(py: Python<'_>, argv: Option<Vec<String>>) -> PyResult<()> {
    let argv = argv.unwrap_or_default();
    py.allow_threads(move || llm::kv::run_kv_indexer_cli(argv))
        .map_err(kv_indexer_to_pyerr)
}

Ryan Olson's avatar
Ryan Olson committed
232
233
234
235
236
237
238
/// Log a message from Python with file and line info
#[pyfunction]
#[pyo3(text_signature = "(level, message, module, file, line)")]
fn log_message(level: &str, message: &str, module: &str, file: &str, line: u32) {
    logging::log_message(level, message, module, file, line);
}

239
240
241
242
243
244
245
/// Generate a deterministic signed int32 ID from a LoRA name using blake3 hash.
#[pyfunction]
#[pyo3(text_signature = "(lora_name)")]
fn lora_name_to_id(lora_name: &str) -> i32 {
    llm_rs::utils::lora_name_to_id(lora_name)
}

246
247
/// Create an engine and attach it to an endpoint to make it visible to the frontend.
/// This is the main way you create a Dynamo worker / backend.
248
249
250
251
252
253
254
///
/// If `lora_name` is provided, this function will publish a LoRA adapter instead of a base model:
/// - LoRA path: v1/mdc/{namespace}/{component}/{endpoint}/{instance_id}/{lora_slug}
/// - Base model path: v1/mdc/{namespace}/{component}/{endpoint}/{instance_id}
///
/// For LoRA mode, both `lora_name` and `base_model_path` must be provided together.
/// Providing only one of them will result in an error.
255
#[pyfunction]
256
#[pyo3(signature = (model_input, model_type, endpoint, model_path, model_name=None, context_length=None, kv_cache_block_size=None, router_mode=None, runtime_config=None, user_data=None, custom_template_path=None, media_decoder=None, media_fetcher=None, lora_name=None, base_model_path=None))]
257
#[allow(clippy::too_many_arguments)]
258
fn register_model<'p>(
259
    py: Python<'p>,
260
    model_input: ModelInput,
261
    model_type: ModelType,
262
263
264
    endpoint: Endpoint,
    model_path: &str,
    model_name: Option<&str>,
265
266
    context_length: Option<u32>,
    kv_cache_block_size: Option<u32>,
267
    router_mode: Option<RouterMode>,
268
    runtime_config: Option<ModelRuntimeConfig>,
269
    user_data: Option<&Bound<'p, PyDict>>,
270
    custom_template_path: Option<&str>,
271
272
    media_decoder: Option<MediaDecoder>,
    media_fetcher: Option<MediaFetcher>,
273
274
    lora_name: Option<&str>,
    base_model_path: Option<&str>,
275
) -> PyResult<Bound<'p, PyAny>> {
276
    // Validate Prefill model type requirements
277
278
279
280
281
282
    if model_type.inner == llm_rs::model_type::ModelType::Prefill
        && !matches!(model_input, ModelInput::Tokens)
    {
        return Err(PyErr::new::<pyo3::exceptions::PyValueError, _>(
            "ModelType::Prefill requires model_input to be ModelInput::Tokens",
        ));
283
284
    }

285
286
287
    let model_input = match model_input {
        ModelInput::Text => llm_rs::model_type::ModelInput::Text,
        ModelInput::Tokens => llm_rs::model_type::ModelInput::Tokens,
288
        ModelInput::Tensor => llm_rs::model_type::ModelInput::Tensor,
289
290
    };

291
    let is_tensor_based = model_type.inner.supports_tensor();
292
    let is_images = model_type.inner.supports_images();
293
    let is_videos = model_type.inner.supports_videos();
294

295
296
    let model_type_obj = model_type.inner;

297
    let inner_path = model_path.to_string();
298
    let model_name = model_name.map(|n| n.to_string());
299
300
301
    let router_mode = router_mode.unwrap_or(RouterMode::RoundRobin);
    let router_config = RouterConfig::new(router_mode.into(), KvRouterConfig::default());

302
303
304
305
306
307
308
309
310
311
312
313
314
    // Early validation of custom template path
    let custom_template_path_owned = custom_template_path
        .map(|s| {
            let path = PathBuf::from(s);
            if !path.exists() {
                return Err(PyErr::new::<pyo3::exceptions::PyFileNotFoundError, _>(
                    format!("Custom template file does not exist: {}", path.display()),
                ));
            }
            Ok(path)
        })
        .transpose()?;

315
316
317
318
319
320
321
    let user_data_json = user_data
        .map(|dict| pythonize::depythonize(dict))
        .transpose()
        .map_err(|err| {
            PyErr::new::<PyException, _>(format!("Failed to convert user_data: {}", err))
        })?;

322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
    // Validate LoRA parameters: both or neither must be provided
    if lora_name.is_some() ^ base_model_path.is_some() {
        return Err(PyErr::new::<pyo3::exceptions::PyValueError, _>(
            "lora_name and base_model_path must both be provided together, or neither",
        ));
    }

    // Determine source_path and lora_identifier based on registration mode
    let (source_path, lora_identifier) = match (lora_name, base_model_path) {
        (Some(lora), Some(base)) => (base.to_string(), Some(lora.to_string())),
        _ => (inner_path, None),
    };

    // Model name: use lora name if present, otherwise provided name or default to source path
    let model_name = lora_identifier
        .clone()
        .or(model_name)
        .or_else(|| Some(source_path.clone()));

341
    pyo3_async_runtimes::tokio::future_into_py(py, async move {
342
343
344
        // For TensorBased, Images, and Videos models, skip HuggingFace downloads and register directly
        // These model types handle model loading internally, no tokenizer extraction needed
        if is_tensor_based || is_images || is_videos {
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
            let model_name = model_name.unwrap_or_else(|| source_path.clone());
            let mut card = llm_rs::model_card::ModelDeploymentCard::with_name_only(&model_name);
            card.model_type = model_type_obj;
            card.model_input = model_input;
            card.user_data = user_data_json;

            if let Some(cfg) = runtime_config {
                card.runtime_config = cfg.inner;
            }

            // Register the Model Deployment Card via discovery interface
            let discovery = endpoint.inner.drt().discovery();
            let spec = rs::discovery::DiscoverySpec::from_model(
                endpoint.inner.component().namespace().name().to_string(),
                endpoint.inner.component().name().to_string(),
                endpoint.inner.name().to_string(),
                &card,
            )
            .map_err(to_pyerr)?;
            discovery.register(spec).await.map_err(to_pyerr)?;

            return Ok(());
        }

        // For non-TensorBased models, resolve the model path (local or fetch from HuggingFace)
370
371
        let model_path = if fs::exists(&source_path)? {
            PathBuf::from(&source_path)
372
        } else {
373
            LocalModel::fetch(&source_path, false)
374
375
376
377
                .await
                .map_err(to_pyerr)?
        };

378
379
        let mut builder = dynamo_llm::local_model::LocalModelBuilder::default();
        builder
380
            // model path is the physical path on disk of the downloaded model
381
            .model_path(model_path)
382
383
384
385
            // source path is what the user gave as `--model-path`, either a real path (in which
            // case it matches model_path above), or an HF repo.
            .source_path(source_path.clone().into())
            // --served_model_name
386
            .model_name(model_name.clone())
387
            .context_length(context_length)
388
            .kv_cache_block_size(kv_cache_block_size)
389
            .router_config(Some(router_config))
390
            .runtime_config(runtime_config.unwrap_or_default().inner)
391
            .user_data(user_data_json)
392
393
394
            .custom_template_path(custom_template_path_owned)
            .media_decoder(media_decoder.map(|m| m.inner))
            .media_fetcher(media_fetcher.map(|m| m.inner));
395

396
        let mut local_model = builder.build().await.map_err(to_pyerr)?;
397
398
399
400
401
402
403
404
405

        // Convert lora_identifier (Option<String>) to Option<LoraInfo>
        let lora_info = lora_identifier
            .as_ref()
            .map(|name| llm_rs::model_card::LoraInfo {
                name: name.clone(),
                max_gpu_lora_count: None,
            });

406
        local_model
407
            .attach(&endpoint.inner, model_type_obj, model_input, lora_info)
408
409
410
            .await
            .map_err(to_pyerr)?;

411
412
413
        if let Some(lora_name) = lora_identifier {
            tracing::info!("Registered LoRA '{}' MDC", lora_name);
        } else {
414
415
416
417
            tracing::info!(
                "Registered base model '{}' MDC",
                model_name.unwrap_or(source_path)
            );
418
419
        }

420
421
422
423
        Ok(())
    })
}

424
425
426
427
428
429
430
431
432
433
434
435
436
/// Unregister a Model Deployment Card (MDC) from the service registry
///
/// This removes an LLM deployment from the discovery system.
///
/// # Arguments
///
/// * `endpoint` - The endpoint where the model is registered
/// * `lora_name` - Optional LoRA adapter name (if unregistering a LoRA deployment)
///
/// # MDC Path Format
///
/// - Base model: `v1/mdc/{namespace}/{component}/{endpoint}/{instance_id}`
/// - LoRA model: `v1/mdc/{namespace}/{component}/{endpoint}/{instance_id}/{lora_slug}`
437
#[pyfunction]
438
#[pyo3(signature = (endpoint, lora_name=None))]
439
fn unregister_model<'p>(
440
441
442
443
444
445
    py: Python<'p>,
    endpoint: Endpoint,
    lora_name: Option<&str>,
) -> PyResult<Bound<'p, PyAny>> {
    let lora_name_owned = lora_name.map(|s| s.to_string());

446
    pyo3_async_runtimes::tokio::future_into_py(py, async move {
447
448
        // Unified detach method handles both base models and LoRA adapters
        LocalModel::detach_from_endpoint(&endpoint.inner, lora_name_owned.as_deref())
449
450
451
452
453
454
            .await
            .map_err(to_pyerr)?;
        Ok(())
    })
}

455
456
/// Download a model from Hugging Face, returning its local path
/// Example: `model_path = await fetch_model("Qwen/Qwen3-0.6B")`
457
#[pyfunction]
458
#[pyo3(signature = (remote_name, ignore_weights=false))]
459
fn fetch_model<'p>(
460
461
462
463
    py: Python<'p>,
    remote_name: &str,
    ignore_weights: bool,
) -> PyResult<Bound<'p, PyAny>> {
464
465
    let repo = remote_name.to_string();
    pyo3_async_runtimes::tokio::future_into_py(py, async move {
466
467
468
        LocalModel::fetch(&repo, ignore_weights)
            .await
            .map_err(to_pyerr)
469
470
471
    })
}

472
473
#[pyclass]
#[derive(Clone)]
Ryan Olson's avatar
Ryan Olson committed
474
pub struct DistributedRuntime {
475
476
477
478
    inner: rs::DistributedRuntime,
    event_loop: PyObject,
}

Ryan Olson's avatar
Ryan Olson committed
479
480
impl DistributedRuntime {
    #[allow(dead_code)]
481
    pub(crate) fn inner(&self) -> &rs::DistributedRuntime {
Ryan Olson's avatar
Ryan Olson committed
482
483
484
485
        &self.inner
    }
}

486
#[pyclass]
487
#[derive(Clone)]
488
489
490
491
492
struct CancellationToken {
    inner: rs::CancellationToken,
}

#[pyclass]
493
#[derive(Clone)]
494
495
496
497
498
struct Endpoint {
    inner: rs::component::Endpoint,
    event_loop: PyObject,
}

499
500
501
502
503
504
#[pyclass]
#[derive(Clone)]
struct ModelCardInstanceId {
    inner: rs::discovery::ModelCardInstanceId,
}

505
#[pyclass]
506
#[derive(Clone)]
507
struct Client {
508
    router: rs::pipeline::PushRouter<serde_json::Value, RsAnnotated<serde_json::Value>>,
509
510
}

511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
#[pyclass]
#[derive(Clone, PartialEq)]
struct ModelType {
    inner: llm_rs::model_type::ModelType,
}

#[pymethods]
#[allow(non_upper_case_globals)]
impl ModelType {
    #[classattr]
    const Chat: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Chat,
    };
    #[classattr]
    const Completions: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Completions,
    };
    #[classattr]
    const Embedding: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Embedding,
    };
532
533
534
535
    #[classattr]
    const TensorBased: Self = ModelType {
        inner: llm_rs::model_type::ModelType::TensorBased,
    };
536
537
538
539
    #[classattr]
    const Prefill: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Prefill,
    };
540
541
542
543
    #[classattr]
    const Images: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Images,
    };
544
    #[classattr]
545
546
547
548
    const Audios: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Audios,
    };
    #[classattr]
549
550
551
    const Videos: Self = ModelType {
        inner: llm_rs::model_type::ModelType::Videos,
    };
552

553
554
555
556
    fn supports_chat(&self) -> bool {
        self.inner.supports_chat()
    }

557
558
559
560
561
562
563
564
565
566
567
    fn __or__(&self, other: &Self) -> Self {
        ModelType {
            inner: self.inner | other.inner,
        }
    }

    fn __str__(&self) -> String {
        self.inner.to_string()
    }
}

568
569
#[pyclass(eq, eq_int)]
#[derive(Clone, PartialEq)]
570
571
572
enum ModelInput {
    Text = 1,
    Tokens = 2,
573
    Tensor = 3,
574
575
}

576
577
578
#[pymethods]
impl DistributedRuntime {
    #[new]
579
    #[pyo3(signature = (event_loop, discovery_backend, request_plane, enable_nats=None))]
580
581
    fn new(
        event_loop: PyObject,
582
        discovery_backend: String,
583
584
585
        request_plane: String,
        enable_nats: Option<bool>,
    ) -> PyResult<Self> {
586
587
588
589
590
591
592
        let discovery_backend_config = match discovery_backend.as_str() {
            "kubernetes" => DiscoveryBackend::Kubernetes,
            other => {
                let selector: kv::Selector = other.parse().map_err(to_pyerr)?;
                DiscoveryBackend::KvStore(selector)
            }
        };
593
        let request_plane: RequestPlaneMode = request_plane.parse().map_err(to_pyerr)?;
594

595
596
597
        // Try to get existing runtime first, create new Worker only if needed
        // This allows multiple DistributedRuntime instances to share the same tokio runtime
        let runtime = rs::Worker::runtime_from_existing()
598
            .or_else(|_| -> anyhow::Result<rs::Runtime> {
599
600
601
602
                // No existing Worker, create new one
                let worker = rs::Worker::from_settings()?;

                // Initialize pyo3 bridge (only happens once per process)
603
                INIT.get_or_try_init(|| -> anyhow::Result<()> {
604
605
                    let primary = worker.tokio_runtime()?;
                    pyo3_async_runtimes::tokio::init_with_runtime(primary).map_err(|e| {
606
                        anyhow::anyhow!("failed to initialize pyo3 static runtime: {:?}", e)
607
                    })?;
608
                    Ok(())
609
610
                })?;

611
                Ok(worker.runtime().clone())
612
613
            })
            .map_err(to_pyerr)?;
614

615
616
        // Initialize logging in context where tokio runtime is available
        // otel exporter requires it
617
        if config::env_is_truthy(env_otlp::OTEL_EXPORT_ENABLED) {
618
619
620
621
            runtime.secondary().block_on(async {
                rs::logging::init();
            });
        }
622

623
624
625
626
627
628
629
630
631
632
        // NATS is used for more than just the NATS request-plane:
        // - KV router events (JetStream or NATS core + local indexer)
        // - inter-router replica sync (NATS core)
        //
        // NATS initialization logic:
        // 1. If request_plane is NATS, always enable NATS
        // 2. Otherwise, use enable_nats parameter (defaults to true for backward compat)
        //    Pass false to disable NATS (e.g., for approximate KV routing mode)
        let enable_nats = enable_nats.unwrap_or(true); // Default to true

633
        let runtime_config = DistributedConfig {
634
            discovery_backend: discovery_backend_config,
635
            nats_config: if request_plane.is_nats() || enable_nats {
636
637
638
639
                Some(dynamo_runtime::transports::nats::ClientOptions::default())
            } else {
                None
            },
640
            request_plane,
641
642
643
644
645
        };
        let inner = runtime
            .secondary()
            .block_on(rs::DistributedRuntime::new(runtime, runtime_config))
            .map_err(to_pyerr)?;
646
647
648
649

        Ok(DistributedRuntime { inner, event_loop })
    }

Ryan Olson's avatar
Ryan Olson committed
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
    #[staticmethod]
    fn detached(py: Python) -> PyResult<Self> {
        let rt = rs::Worker::runtime_from_existing().map_err(to_pyerr)?;
        let handle = rt.primary();

        let inner = handle
            .block_on(rs::DistributedRuntime::from_settings(rt))
            .map_err(to_pyerr)?;

        Ok(DistributedRuntime {
            inner,
            event_loop: py.None(),
        })
    }

665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
    /// Get an endpoint directly by path (e.g., "namespace.component.endpoint" or "dyn://...").
    fn endpoint(&self, path: String) -> PyResult<Endpoint> {
        let trimmed_path = path.trim_start_matches("dyn://");
        let parts: Vec<&str> = trimmed_path.split('.').collect();

        if parts.len() != 3 {
            return Err(PyErr::new::<pyo3::exceptions::PyValueError, _>(format!(
                "Invalid endpoint path '{}'. Expected format: 'namespace.component.endpoint' or 'dyn://namespace.component.endpoint'",
                path
            )));
        }

        let namespace_name = parts[0];
        let component_name = parts[1];
        let endpoint_name = parts[2];

        // Get endpoint using existing chain
        let namespace = self
            .inner
            .namespace(namespace_name.to_string())
            .map_err(to_pyerr)?;
        let component = namespace
            .component(component_name.to_string())
            .map_err(to_pyerr)?;
        let endpoint = component.endpoint(endpoint_name.to_string());

        Ok(Endpoint {
            inner: endpoint,
693
694
695
696
697
            event_loop: self.event_loop.clone(),
        })
    }

    fn shutdown(&self) {
698
        self.inner.shutdown();
699
700
701
702
703
    }

    fn event_loop(&self) -> PyObject {
        self.event_loop.clone()
    }
704

705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
    /// Register an async Python callback for /engine/{route_name}
    ///
    /// Args:
    ///     route_name: Route path (e.g., "start_profile" → /engine/start_profile)
    ///     callback: Async function with signature: async def(body: dict) -> dict
    ///
    /// Example:
    /// ```python
    /// async def start_profile(body: dict) -> dict:
    ///     await engine.start_profile(**body)
    ///     return {"status": "ok"}
    ///
    /// runtime.register_engine_route("start_profile", start_profile)
    /// ```
    #[pyo3(signature = (route_name, callback))]
    fn register_engine_route(
        &self,
        py: Python<'_>,
        route_name: String,
        callback: PyObject,
    ) -> PyResult<()> {
        // Capture TaskLocals at registration time when Python's event loop is running.
        // This is needed because later, when the callback is invoked from an HTTP request,
        // we'll be on a Rust thread without a running Python event loop.
        let locals =
            Arc::new(pyo3_async_runtimes::tokio::get_current_locals(py).map_err(to_pyerr)?);
        let callback = Arc::new(callback);

        // Wrap Python async callback in Rust async closure
        let rust_callback: rs::engine_routes::EngineRouteCallback =
            Arc::new(move |body: serde_json::Value| {
                let callback = callback.clone();
                let locals = locals.clone();

                // Return a boxed future
                Box::pin(async move {
                    // Acquire GIL to call Python callback and convert coroutine to future
                    let py_future = Python::with_gil(|py| {
                        // Convert body to Python dict
                        let py_body = pythonize::pythonize(py, &body).map_err(|e| {
                            anyhow::anyhow!("Failed to convert request body to Python: {}", e)
                        })?;

                        // Call Python async function to get a coroutine
                        let coroutine = callback.call1(py, (py_body,)).map_err(|e| {
                            anyhow::anyhow!("Failed to call Python callback: {}", e)
                        })?;

                        // Use the TaskLocals captured at registration time
                        pyo3_async_runtimes::into_future_with_locals(
                            &locals,
                            coroutine.into_bound(py),
                        )
                        .map_err(|e| {
                            anyhow::anyhow!("Failed to convert coroutine to future: {}", e)
                        })
                    })?;

                    // Await the Python coroutine (GIL is released during await)
                    let py_result = py_future
                        .await
                        .map_err(|e| anyhow::anyhow!("Python callback failed: {}", e))?;

                    // Convert result back to serde_json::Value
                    Python::with_gil(|py| {
                        pythonize::depythonize::<serde_json::Value>(py_result.bind(py))
                            .map_err(|e| anyhow::anyhow!("Failed to serialize response: {}", e))
                    })
                })
            });

        self.inner
            .engine_routes()
            .register(&route_name, rust_callback);
        tracing::debug!("Registered engine route: /engine/{}", route_name);
        Ok(())
    }

783
784
785
786
787
788
789
790
791
792
793
    /// Set the system-level health status (Ready / NotReady).
    fn set_health_status(&self, ready: bool) -> PyResult<()> {
        let status = if ready {
            config::HealthStatus::Ready
        } else {
            config::HealthStatus::NotReady
        };
        self.inner.system_health().lock().set_health_status(status);
        Ok(())
    }

Richard Huo's avatar
Richard Huo committed
794
795
796
797
798
799
800
801
802
803
804
805
806
807
    // This is used to pass the DistributedRuntime from the dynamo-runtime bindings
    // to the KVBM bindings, since KVBM cannot directly use the struct from this cdylib.
    // TODO: Create a separate crate "dynamo-python" so that all binding crates can import
    // from it and share the same crate path. This will allow PyO3 to automatically
    // recognize that both bindings use the same PyClass.
    #[pyo3(name = "to_capsule")]
    fn to_capsule<'py>(&self, py: Python<'py>) -> PyResult<Bound<'py, PyCapsule>> {
        let arc: Arc<rs::DistributedRuntime> = Arc::new(self.inner.clone());
        let weak: Weak<rs::DistributedRuntime> = Arc::downgrade(&arc);

        let name = CString::new("dynamo.runtime.weak").expect("valid capsule name");

        PyCapsule::new(py, weak, Some(name))
    }
808
809
810
811
}

#[pymethods]
impl Endpoint {
812
    #[pyo3(signature = (generator, graceful_shutdown = true, metrics_labels = None, health_check_payload = None))]
813
814
815
816
    fn serve_endpoint<'p>(
        &self,
        py: Python<'p>,
        generator: PyObject,
817
        graceful_shutdown: Option<bool>,
818
        metrics_labels: Option<Vec<(String, String)>>,
819
        health_check_payload: Option<&Bound<'p, PyDict>>,
820
821
822
823
824
    ) -> PyResult<Bound<'p, PyAny>> {
        let engine = Arc::new(engine::PythonAsyncEngine::new(
            generator,
            self.event_loop.clone(),
        )?);
825
        let ingress = JsonServerStreamingIngress::for_engine(engine.clone()).map_err(to_pyerr)?;
826
827
828
829
830
831
832
833
834
835
836
837
838

        // Convert Python dict to serde_json::Value if provided and validate it's an object
        let health_payload_json = health_check_payload
            .map(|dict| pythonize::depythonize::<serde_json::Value>(dict))
            .transpose()
            .map_err(|err| {
                pyo3::exceptions::PyTypeError::new_err(format!(
                    "Failed to convert health_check_payload: {}",
                    err
                ))
            })?;

        // Require an object/dict
839
840
841
842
843
844
        if let Some(ref payload) = health_payload_json
            && !payload.is_object()
        {
            return Err(pyo3::exceptions::PyTypeError::new_err(
                "health_check_payload must be a JSON object (dict)",
            ));
845
846
847
        }

        let mut builder = self
848
849
850
851
            .inner
            .endpoint_builder()
            .metrics_labels(metrics_labels)
            .handler(ingress);
852
853
854
855
856

        if let Some(payload) = health_payload_json {
            builder = builder.health_check_payload(payload);
        }

857
858
859
        // Register the engine in the local endpoint registry for in-process calls
        builder = builder.register_local_engine(engine).map_err(to_pyerr)?;

860
        let graceful_shutdown = graceful_shutdown.unwrap_or(true);
861
        pyo3_async_runtimes::tokio::future_into_py(py, async move {
862
863
864
865
866
            builder
                .graceful_shutdown(graceful_shutdown)
                .start()
                .await
                .map_err(to_pyerr)?;
867
868
869
870
            Ok(())
        })
    }

871
872
873
874
875
876
877
    #[pyo3(signature = (router_mode = None))]
    fn client<'p>(
        &self,
        py: Python<'p>,
        router_mode: Option<RouterMode>,
    ) -> PyResult<Bound<'p, PyAny>> {
        let router_mode = router_mode.unwrap_or(RouterMode::RoundRobin);
878
879
        let inner = self.inner.clone();
        pyo3_async_runtimes::tokio::future_into_py(py, async move {
880
            let client = inner.client().await.map_err(to_pyerr)?;
881
882
883
            let push_router = rs::pipeline::PushRouter::<
                serde_json::Value,
                RsAnnotated<serde_json::Value>,
884
            >::from_client(client, router_mode.into())
885
886
            .await
            .map_err(to_pyerr)?;
887
888
889
            Ok(Client {
                router: push_router,
            })
890
891
        })
    }
892

893
894
895
    // Opaque unique ID for this worker. May change over worker lifetime.
    fn connection_id(&self) -> u64 {
        self.inner.drt().connection_id()
896
    }
897
898
899

    /// Get a RuntimeMetrics helper for creating Prometheus metrics
    #[getter]
900
901
    fn metrics(&self) -> prometheus_metrics::RuntimeMetrics {
        prometheus_metrics::RuntimeMetrics::from_endpoint(self.inner.clone())
902
    }
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931

    /// Unregister this endpoint instance from discovery.
    ///
    /// This removes the endpoint from the instances bucket, preventing the router
    /// from sending requests to this worker. Use this when a worker is sleeping
    /// and should not receive any requests.
    fn unregister_endpoint_instance<'p>(&self, py: Python<'p>) -> PyResult<Bound<'p, PyAny>> {
        let inner = self.inner.clone();
        pyo3_async_runtimes::tokio::future_into_py(py, async move {
            inner
                .unregister_endpoint_instance()
                .await
                .map_err(to_pyerr)?;
            Ok(())
        })
    }

    /// Re-register this endpoint instance to discovery.
    ///
    /// This adds the endpoint back to the instances bucket, allowing the router
    /// to send requests to this worker again. Use this when a worker wakes up
    /// and should start receiving requests.
    fn register_endpoint_instance<'p>(&self, py: Python<'p>) -> PyResult<Bound<'p, PyAny>> {
        let inner = self.inner.clone();
        pyo3_async_runtimes::tokio::future_into_py(py, async move {
            inner.register_endpoint_instance().await.map_err(to_pyerr)?;
            Ok(())
        })
    }
932
933
}

934
935
936
937
938
939
940
941
942
943
944
945
946
#[pymethods]
impl ModelCardInstanceId {
    // (namespace, component, endpoint)
    // TODO: Can these be borrowed as &str?
    fn triple(&self) -> (String, String, String) {
        (
            self.inner.namespace.clone(),
            self.inner.component.clone(),
            self.inner.endpoint.clone(),
        )
    }
}

947
948
#[pymethods]
impl Client {
949
950
    /// Get list of current instances.
    /// Replaces endpoint_ids.
951
    fn instance_ids(&self) -> Vec<u64> {
952
        self.router.client.instance_ids()
953
954
    }

955
956
957
    /// Wait for an instance to be available for work.
    /// Replaces wait_for_endpoints.
    fn wait_for_instances<'p>(&self, py: Python<'p>) -> PyResult<Bound<'p, PyAny>> {
958
        let inner = self.router.client.clone();
959
        pyo3_async_runtimes::tokio::future_into_py(py, async move {
960
            inner
961
                .wait_for_instances()
962
                .await
963
                .map(|v| v.into_iter().map(|cei| cei.id()).collect::<Vec<u64>>())
964
                .map_err(to_pyerr)
965
966
967
968
        })
    }

    /// Issue a request to the endpoint using the default routing strategy.
969
    #[pyo3(signature = (request, annotated=DEFAULT_ANNOTATED_SETTING, context=None))]
970
971
972
973
974
    fn generate<'p>(
        &self,
        py: Python<'p>,
        request: PyObject,
        annotated: Option<bool>,
975
        context: Option<context::Context>,
976
    ) -> PyResult<Bound<'p, PyAny>> {
977
        self.random(py, request, annotated, context)
978
979
980
    }

    /// Send a request to the next endpoint in a round-robin fashion.
981
    #[pyo3(signature = (request, annotated=DEFAULT_ANNOTATED_SETTING, context=None))]
982
983
984
985
986
    fn round_robin<'p>(
        &self,
        py: Python<'p>,
        request: PyObject,
        annotated: Option<bool>,
987
        context: Option<context::Context>,
988
989
    ) -> PyResult<Bound<'p, PyAny>> {
        let request: serde_json::Value = pythonize::depythonize(&request.into_bound(py))?;
990
        let request_ctx = create_request_context(request, &context);
991
992
993
        let annotated = annotated.unwrap_or(false);

        let (tx, rx) = tokio::sync::mpsc::channel(32);
994
        let client = self.router.clone();
995
996

        pyo3_async_runtimes::tokio::future_into_py(py, async move {
997
998
            let stream = match context {
                Some(context) => {
999
1000
                    // Always instrument with appropriate span (none if no trace context)
                    let span = get_span_for_context(&context, "round_robin");
1001
1002
1003
1004
1005
                    client
                        .round_robin(request_ctx)
                        .instrument(span)
                        .await
                        .map_err(to_pyerr)?
1006
                }
1007
                _ => client.round_robin(request_ctx).await.map_err(to_pyerr)?,
1008
            };
1009
            tokio::spawn(process_stream(stream, tx));
1010
            Ok(AsyncResponseStream::new(rx, annotated))
1011
1012
1013
1014
        })
    }

    /// Send a request to a random endpoint.
1015
    #[pyo3(signature = (request, annotated=DEFAULT_ANNOTATED_SETTING, context=None))]
1016
1017
1018
1019
1020
    fn random<'p>(
        &self,
        py: Python<'p>,
        request: PyObject,
        annotated: Option<bool>,
1021
        context: Option<context::Context>,
1022
1023
    ) -> PyResult<Bound<'p, PyAny>> {
        let request: serde_json::Value = pythonize::depythonize(&request.into_bound(py))?;
1024
        let request_ctx = create_request_context(request, &context);
1025
1026
1027
        let annotated = annotated.unwrap_or(false);

        let (tx, rx) = tokio::sync::mpsc::channel(32);
1028
        let client = self.router.clone();
1029
1030

        pyo3_async_runtimes::tokio::future_into_py(py, async move {
1031
1032
            let stream = match context {
                Some(context) => {
1033
                    // Always instrument with appropriate span (none if no trace context)
1034
                    let span = get_span_for_context(&context, "random");
1035
1036
1037
1038
1039
                    client
                        .random(request_ctx)
                        .instrument(span)
                        .await
                        .map_err(to_pyerr)?
1040
                }
1041
                _ => client.random(request_ctx).await.map_err(to_pyerr)?,
1042
            };
1043
            tokio::spawn(process_stream(stream, tx));
1044
            Ok(AsyncResponseStream::new(rx, annotated))
1045
1046
1047
1048
        })
    }

    /// Directly send a request to a specific endpoint.
1049
    #[pyo3(signature = (request, instance_id, annotated=DEFAULT_ANNOTATED_SETTING, context=None))]
1050
1051
1052
1053
    fn direct<'p>(
        &self,
        py: Python<'p>,
        request: PyObject,
1054
        instance_id: u64,
1055
        annotated: Option<bool>,
1056
        context: Option<context::Context>,
1057
1058
    ) -> PyResult<Bound<'p, PyAny>> {
        let request: serde_json::Value = pythonize::depythonize(&request.into_bound(py))?;
1059
        let request_ctx = create_request_context(request, &context);
1060
1061
1062
        let annotated = annotated.unwrap_or(false);

        let (tx, rx) = tokio::sync::mpsc::channel(32);
1063
        let client = self.router.clone();
1064
1065

        pyo3_async_runtimes::tokio::future_into_py(py, async move {
1066
1067
            let stream = match context {
                Some(context) => {
1068
                    // Always instrument with appropriate span (none if no trace context)
1069
1070
                    let span =
                        get_span_for_direct_context(&context, "direct", &instance_id.to_string());
1071
1072
1073
1074
1075
                    client
                        .direct(request_ctx, instance_id)
                        .instrument(span)
                        .await
                        .map_err(to_pyerr)?
1076
1077
                }
                _ => client
1078
                    .direct(request_ctx, instance_id)
1079
1080
1081
                    .await
                    .map_err(to_pyerr)?,
            };
1082
1083
1084

            tokio::spawn(process_stream(stream, tx));

1085
            Ok(AsyncResponseStream::new(rx, annotated))
1086
1087
1088
1089
1090
        })
    }
}

async fn process_stream(
1091
    stream: EngineStream<RsAnnotated<serde_json::Value>>,
1092
1093
1094
1095
1096
    tx: tokio::sync::mpsc::Sender<RsAnnotated<PyObject>>,
) {
    let mut stream = stream;
    while let Some(response) = stream.next().await {
        // Convert the response to a PyObject using Python's GIL
1097
        let annotated: RsAnnotated<serde_json::Value> = response;
1098
        let annotated: RsAnnotated<PyObject> = annotated.map_data(|data| {
1099
            Python::with_gil(|py| match pythonize::pythonize(py, &data) {
1100
1101
                Ok(pyobj) => Ok(pyobj.into()),
                Err(e) => Err(e.to_string()),
1102
            })
1103
1104
1105
1106
1107
1108
        });

        let is_error = annotated.is_error();

        // Send the PyObject through the channel or log an error
        if let Err(e) = tx.send(annotated).await {
1109
            tracing::error!("Failed to send response: {:?}", e);
1110
            break;
1111
1112
1113
1114
1115
1116
1117
1118
1119
        }

        if is_error {
            break;
        }
    }
}

#[pyclass]
1120
pub(crate) struct AsyncResponseStream {
1121
1122
1123
    rx: Arc<Mutex<tokio::sync::mpsc::Receiver<RsAnnotated<PyObject>>>>,
    annotated: bool,
}
1124
1125
1126
1127
1128
1129
1130
1131
1132
1133
1134
1135

impl AsyncResponseStream {
    pub(crate) fn new(
        rx: tokio::sync::mpsc::Receiver<RsAnnotated<PyObject>>,
        annotated: bool,
    ) -> Self {
        Self {
            rx: Arc::new(Mutex::new(rx)),
            annotated,
        }
    }
}
1136
1137
1138
1139
1140
1141
1142
1143
1144
1145
1146
1147
1148
1149
1150
1151
1152
1153
1154
1155
1156
1157
1158
1159
1160
1161
1162
1163
1164
1165
1166
1167
1168
1169
1170
1171
1172
1173
1174
1175
1176
1177
1178
1179
1180
1181
1182
1183
1184
1185
1186
1187
1188
1189
1190
1191
1192
1193
1194
1195
1196
1197
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
1220
1221
1222
1223
1224
1225
1226
1227
1228
1229
1230
1231
1232

#[pymethods]
impl AsyncResponseStream {
    /// This method is required to implement the `AsyncIterator` protocol.
    #[pyo3(name = "__aiter__")]
    fn aiter(slf: PyRef<Self>, py: Python) -> PyResult<Py<PyAny>> {
        slf.into_py_any(py)
    }
    /// This method is required to implement the `AsyncIterator` protocol.
    #[pyo3(name = "__anext__")]
    fn next<'p>(&self, py: Python<'p>) -> PyResult<Bound<'p, PyAny>> {
        let rx = self.rx.clone();
        let annotated = self.annotated;

        pyo3_async_runtimes::tokio::future_into_py(py, async move {
            loop {
                let value = rx.lock().await.recv().await;
                match value {
                    Some(pyobj) => {
                        let pyobj = match pyobj.ok() {
                            Ok(pyobj) => pyobj,
                            Err(e) => {
                                return Err(PyErr::new::<pyo3::exceptions::PyValueError, _>(e));
                            }
                        };

                        if annotated {
                            let object = Annotated { inner: pyobj };
                            #[allow(deprecated)]
                            let object = Python::with_gil(|py| object.into_py(py));
                            return Ok(object);
                        } else {
                            match pyobj.data {
                                Some(data) => return Ok(data),
                                None => continue,
                            }
                        }
                    }
                    None => return Err(PyStopAsyncIteration::new_err("Stream exhausted")),
                }
            }
        })
    }
}

#[pyclass]
struct Annotated {
    inner: RsAnnotated<PyObject>,
}

#[pymethods]
impl Annotated {
    #[new]
    fn new(data: PyObject) -> Self {
        Annotated {
            inner: RsAnnotated::from_data(data),
        }
    }

    fn is_error(&self) -> bool {
        self.inner.is_error()
    }

    fn data(&self) -> Option<PyObject> {
        self.inner.data.clone()
    }

    fn event(&self) -> Option<String> {
        self.inner.event.clone()
    }

    fn comments(&self) -> Option<Vec<String>> {
        self.inner.comment.clone()
    }

    fn id(&self) -> Option<String> {
        self.inner.id.clone()
    }

    #[pyo3(name = "__repr__")]
    fn _repr(&self, py: Python) -> String {
        let data = self.inner.data.clone().map(|obj| {
            obj.call_method0(py, "__repr__")
                .and_then(|repr_obj| repr_obj.extract::<Py<PyString>>(py))
                .map(|py_str| py_str.to_string_lossy(py).into_owned())
                .unwrap_or_else(|_| "<failed_repr>".to_string())
        });

        format!(
            "Annotated(data={}, event={}, comment={:?}, id={})",
            data.unwrap_or_else(|| "<no_data>".to_string()),
            self.inner.event.as_deref().unwrap_or("None"),
            self.inner.comment.as_deref().unwrap_or(&[]),
            self.inner.id.as_deref().unwrap_or("None")
        )
    }
}