Skip to content

Stow transcode - #85

Draft
feliwir wants to merge 2 commits into
UMEssen:mainfrom
mbits-imaging:stow-transcode
Draft

feliwir wants to merge 2 commits into
UMEssen:mainfrom
mbits-imaging:stow-transcode

Conversation

@feliwir

@feliwir feliwir commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

We always propose uncompressed transfer-syntaxes (when enabled via options) as a fallback when negotiating an association.

This builds on #84 and fixes #83

pub async fn store(
&self,
mut file: FileDicomObject<InMemDicomObject>,
) -> Result<(), StoreError> {
pub async fn store(
&self,
mut file: FileDicomObject<InMemDicomObject>,
) -> Result<(), StoreError> {
async fn transcode(
mut file: FileDicomObject<InMemDicomObject>,
transfer_syntax_uid: String,
) -> Result<FileDicomObject<InMemDicomObject>, StoreError> {
async fn transcode(
mut file: FileDicomObject<InMemDicomObject>,
transfer_syntax_uid: String,
) -> Result<FileDicomObject<InMemDicomObject>, StoreError> {
Comment on lines +215 to +218
tokio::task::spawn_blocking(move || {
file.transcode(transfer_syntax)?;
Ok(file)
})
Comment on lines +215 to +218
tokio::task::spawn_blocking(move || {
file.transcode(transfer_syntax)?;
Ok(file)
})
Comment thread tests/common/mod.rs

/// Starts Orthanc with the given configuration, which replaces the default `orthanc.json`
/// of the image. The REST API is reachable with the credentials orthanc:orthanc.
pub async fn spawn_orthanc_with_config(

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle unsupported transfer-syntax

2 participants