CinemaDngReader: add support for reading files with missing FPS

apertus-open-source-cinema · Mar 11, 2024 · 6840f12 · 6840f12
1 parent 7f5f88c
commit 6840f12
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/nodes_io/reader_cinema_dng.rs b/src/nodes_io/reader_cinema_dng.rs
@@ -114,10 +114,11 @@ impl ProcessingNode for CinemaDngReader {
             bit_depth: get_tag_as_u32(tags::ifd::BitsPerSample)?,
             fps: dng
                 .get_entry_by_path(&main_ifd.chain_tag(tags::ifd::FrameRate))
-                .ok_or(anyhow!("couldnt read frame rate of DNG {path:?}"))?
-                .value
-                .as_f64()
-                .ok_or(anyhow!("couldnt interpret frame rate of DNG {path:?} as f64"))?,
+                .map(|entry| entry.value.as_f64().ok_or(anyhow!("couldnt interpret frame rate of DNG {path:?} as f64")))
+                .unwrap_or_else(|| {
+                    eprintln!("DNG has no fps, falling back to 24");
+                    Ok(24.0)
+                })?,
             cfa,
         };