Skip to content

Minor improvements#4

Closed
hrueger wants to merge 8 commits into
ybouane:mainfrom
hrueger:updates
Closed

Minor improvements#4
hrueger wants to merge 8 commits into
ybouane:mainfrom
hrueger:updates

Conversation

@hrueger

@hrueger hrueger commented Jun 7, 2025

Copy link
Copy Markdown

Hi @ybouane,
thanks for this very cool library! I'm working on a Camera Characterization Tool for University and this perfectly fits for loading the raw files 🎉

I did have some improvements, feel free to pick what you like:

  • added a rawImageData function to get raw, undebayered sensor data (sorry for the typo in the commit message)
  • the build process now uses submodules instead of cloning the repo, because that makes it easier to have a fixed version of libraw lib and improves the build speed
  • I refactored the directory structure to clean it up a bit
  • I added typescript support
  • I fixed the example (some cors headers adre needed for the SharedArrayBuffer to be available (at least in my testing)
  • I gitignored dist files

I should still add some docs to the readme about the changes in the build process, I think, like initing submodules with git submodule update --init

@hrueger hrueger changed the title feat: add rawImageDataa function Minor improvements Jun 7, 2025
@ybouane

ybouane commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Thanks a lot for this @hrueger, and sorry for the long delay! 🙏

Since this was opened the repo has moved on quite a bit (GPS metadata, thumbnail unpacking, improved TypeScript defs, a LibRaw 0.22.1 rebuild), so the directory restructure / submodule build here now conflicts heavily and largely overlaps with work that landed in the meantime.

Rather than let the good parts get lost, I've reimplemented your most valuable addition — the rawImageData() method for undebayered 16-bit sensor data — on top of the current codebase in #16, with you credited as a contributor. The CORS/COOP-COEP point you raised is also addressed there in the test setup.

Closing this in favor of #16. Really appreciate the contribution! 🎉

@ybouane ybouane closed this Jun 9, 2026
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.

2 participants