-
-
Notifications
You must be signed in to change notification settings - Fork 813
Import ARW files as Images #2106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
8b938a2
to
f4a1042
Compare
f4a1042
to
3007fb6
Compare
Just for context, this is currently stalled due to #1895 (duplicate of #1231). The uncompressed RAW images use a lot of memory. We also appear to clone them a lot (e.g. when running the node graph and running autosave). This exceeds the WASM memory limits leading to a crash whenever you attempt to work with a RAW file. |
And specifically #1604. |
@Keavon Seeing as this code is perfectly fine, I think we should merge this. |
@0HyperCube I doubt that merging this will be useful. This PR only works on very small RAW images which do not exist and after the asset database, this entire PR will need to rewritten anyway. |
7465fad
to
ab724d8
Compare
aa7ff13
to
e11b57a
Compare
c7488cf
to
1875779
Compare
ec51271
to
e025103
Compare
Blocked because of #1895, waiting on #1604 to be unblocked. Closes #2188